Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(11389)

Unified Diff: chrome/browser/resources/settings/people_page/setup_pin_dialog.html

Issue 2822733002: md settings: Small changes to lock screen. (Closed)
Patch Set: Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/settings/people_page/setup_pin_dialog.html
diff --git a/chrome/browser/resources/settings/people_page/setup_pin_dialog.html b/chrome/browser/resources/settings/people_page/setup_pin_dialog.html
index 040c5cbc932ba25c5920f9ee7f196d66a5b87ddf..805c66035ad3df8e5d57010774051ec00ff59ca2 100644
--- a/chrome/browser/resources/settings/people_page/setup_pin_dialog.html
+++ b/chrome/browser/resources/settings/people_page/setup_pin_dialog.html
@@ -30,6 +30,12 @@
#pinKeyboardDiv {
justify-content: center;
}
+
+ /* Use this instead of hidden or dom-if so that the dialog does not resize
+ when the problem appears or disappears. */
+ #problemDiv[invisible] {
+ visibility: hidden;
+ }
stevenjb 2017/04/14 21:47:20 A simpler approach might just be to hide the iron-
sammiequon 2017/04/14 22:11:43 Done.
</style>
<dialog is="cr-dialog" id="dialog" on-close="close"
@@ -39,7 +45,7 @@
<!-- Warning/error; only shown if title is hidden. Id is needed for
tests. -->
<div id="problemDiv" class$="[[problemClass_]] settings-box first"
- hidden$="[[!hasProblem_(problemMessage_)]]">
+ invisible$="[[!hasProblem_(problemMessage_)]]">
<iron-icon icon="settings:warning"></iron-icon>
<span class="middle">[[problemMessage_]]</span>
</div>

Powered by Google App Engine
This is Rietveld 408576698