Commit 569651fe authored by rbpotter's avatar rbpotter Committed by Commit Bot

Print Preview: Adjust more settings padding and set links actionable

Fixing 2 bugs:
(1) When "More settings" is expanded, margin should be 16px
(2) Make links actionable so hand pointer shows up

Bug: 887071, 887228
Change-Id: Iccb185f7e75e6a6fd87eac45b9b0a9f21dae15ca
Reviewed-on: https://chromium-review.googlesource.com/1236501Reviewed-by: default avatarHector Carmona <hcarmona@chromium.org>
Commit-Queue: Rebekah Potter <rbpotter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#592979}
parent 17143592
...@@ -36,7 +36,7 @@ ...@@ -36,7 +36,7 @@
opacity: .65; opacity: .65;
} }
</style> </style>
<div class="link" id="systemDialogLink" <div class="link" id="systemDialogLink" actionable
hidden$="[[!shouldShowSystemDialogLink_]]" hidden$="[[!shouldShowSystemDialogLink_]]"
on-click="onSystemDialogClick_"> on-click="onSystemDialogClick_">
<div class="label">$i18n{systemDialogOption}</div> <div class="label">$i18n{systemDialogOption}</div>
...@@ -50,7 +50,7 @@ ...@@ -50,7 +50,7 @@
class="throbber"></div> class="throbber"></div>
</div> </div>
<if expr="is_macosx"> <if expr="is_macosx">
<div class="link" id="openPdfInPreviewLink" <div class="link" id="openPdfInPreviewLink" actionable
on-click="onOpenInPreviewClick_"> on-click="onOpenInPreviewClick_">
<div class="label">$i18n{openPdfInPreviewOption}</div> <div class="label">$i18n{openPdfInPreviewOption}</div>
<paper-icon-button-light actionable class="icon-external" <paper-icon-button-light actionable class="icon-external"
......
...@@ -17,6 +17,10 @@ ...@@ -17,6 +17,10 @@
pointer-events: none; pointer-events: none;
} }
:host([settings-expanded-by-user]) {
margin-bottom: 16px;
}
div { div {
align-items: center; align-items: center;
border-top: var(--print-preview-settings-border); border-top: var(--print-preview-settings-border);
......
...@@ -10,6 +10,7 @@ Polymer({ ...@@ -10,6 +10,7 @@ Polymer({
properties: { properties: {
settingsExpandedByUser: { settingsExpandedByUser: {
type: Boolean, type: Boolean,
reflectToAttribute: true,
notify: true, notify: true,
}, },
......
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