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

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

Issue 2805383002: MD Settings: Users: Fix add person validation and dialog focus (Closed)
Patch Set: Set invalid properly 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/users_add_user_dialog.html
diff --git a/chrome/browser/resources/settings/people_page/users_add_user_dialog.html b/chrome/browser/resources/settings/people_page/users_add_user_dialog.html
index 6d812fe8da42d04d4b6a8bdf589faeac87207528..69cd20b555f614d515cc6305c2cb010c6e3c45df 100644
--- a/chrome/browser/resources/settings/people_page/users_add_user_dialog.html
+++ b/chrome/browser/resources/settings/people_page/users_add_user_dialog.html
@@ -20,13 +20,15 @@
<div class="title">$i18n{addUsers}</div>
<div class="body">
<paper-input id="addUserInput" label="$i18n{addUsersEmail}" autofocus
- on-input="validate_">
+ on-input="validate_" invalid="[[!isValid_]]">
</paper-input>
</div>
<div class="button-container">
- <paper-button class="cancel-button" on-tap="onCancelTap_"
- id="cancel">$i18n{cancel}</paper-button>
- <paper-button id="add" on-tap="addUser_" class="action-button" disabled>
+ <paper-button class="cancel-button" on-tap="onCancelTap_">
+ $i18n{cancel}
+ </paper-button>
+ <paper-button on-tap="addUser_" class="action-button"
+ disabled$="[[!isValid_]]">
dpapad 2017/04/11 00:03:55 Probably "$=" not needed here, "=" should be suffi
stevenjb 2017/04/11 00:44:22 Either works, Polymer maps it to an attribute, and
$i18n{add}
</paper-button>
</div>

Powered by Google App Engine
This is Rietveld 408576698