Commit a9d55e69 authored by rbpotter's avatar rbpotter Committed by Commit Bot

Print Preview Refresh: Fix cut off cr-input in advanced dialog

Also fix a bug where enter still leads to print if one of the
paper-buttons on the page is focused (should open dialog instead).
Previously print preview did not have paper-buttons, so  did not check
this case.

Bug: 887844, 888465
Change-Id: Ie041ace8ea0fd5d62096331efba727af55e935f5
Reviewed-on: https://chromium-review.googlesource.com/1239365
Commit-Queue: Rebekah Potter <rbpotter@chromium.org>
Reviewed-by: default avatarHector Carmona <hcarmona@chromium.org>
Cr-Commit-Position: refs/heads/master@{#593713}
parent 3e676b46
...@@ -47,8 +47,6 @@ ...@@ -47,8 +47,6 @@
} }
cr-input { cr-input {
height: 28px;
line-height: 24px;
width: 239px; width: 239px;
--cr-input-error-display: none; --cr-input-error-display: none;
} }
......
...@@ -277,7 +277,7 @@ Polymer({ ...@@ -277,7 +277,7 @@ Polymer({
if (e.code == 'Enter' && this.state == print_preview_new.State.READY) { if (e.code == 'Enter' && this.state == print_preview_new.State.READY) {
const activeElementTag = e.path[0].tagName; const activeElementTag = e.path[0].tagName;
if (['BUTTON', 'SELECT', 'A'].includes(activeElementTag)) if (['PAPER-BUTTON', 'BUTTON', 'SELECT', 'A'].includes(activeElementTag))
return; return;
this.onPrintRequested_(); this.onPrintRequested_();
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment