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

Side by Side 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 unified diff | Download patch
OLDNEW
1 <link rel="import" href="chrome://resources/cr_elements/cr_dialog/cr_dialog.html "> 1 <link rel="import" href="chrome://resources/cr_elements/cr_dialog/cr_dialog.html ">
2 <link rel="import" href="chrome://resources/html/assert.html"> 2 <link rel="import" href="chrome://resources/html/assert.html">
3 <link rel="import" href="chrome://resources/html/polymer.html"> 3 <link rel="import" href="chrome://resources/html/polymer.html">
4 <link rel="import" href="chrome://resources/polymer/v1_0/paper-button/paper-butt on.html"> 4 <link rel="import" href="chrome://resources/polymer/v1_0/paper-button/paper-butt on.html">
5 <link rel="import" href="chrome://resources/polymer/v1_0/paper-input/paper-input .html"> 5 <link rel="import" href="chrome://resources/polymer/v1_0/paper-input/paper-input .html">
6 <link rel="import" href="../settings_shared_css.html"> 6 <link rel="import" href="../settings_shared_css.html">
7 7
8 <dom-module id="settings-users-add-user-dialog"> 8 <dom-module id="settings-users-add-user-dialog">
9 <template> 9 <template>
10 <style include="settings-shared"> 10 <style include="settings-shared">
11 paper-input { 11 paper-input {
12 width: var(--paper-input-max-width); 12 width: var(--paper-input-max-width);
13 --paper-input-container-focus-color: var(--google-blue-500); 13 --paper-input-container-focus-color: var(--google-blue-500);
14 --paper-input-container-input: { 14 --paper-input-container-input: {
15 font-size: inherit; 15 font-size: inherit;
16 }; 16 };
17 } 17 }
18 </style> 18 </style>
19 <dialog is="cr-dialog" id="dialog" close-text="$i18n{close}"> 19 <dialog is="cr-dialog" id="dialog" close-text="$i18n{close}">
20 <div class="title">$i18n{addUsers}</div> 20 <div class="title">$i18n{addUsers}</div>
21 <div class="body"> 21 <div class="body">
22 <paper-input id="addUserInput" label="$i18n{addUsersEmail}" autofocus 22 <paper-input id="addUserInput" label="$i18n{addUsersEmail}" autofocus
23 on-input="validate_"> 23 on-input="validate_" invalid="[[!isValid_]]">
24 </paper-input> 24 </paper-input>
25 </div> 25 </div>
26 <div class="button-container"> 26 <div class="button-container">
27 <paper-button class="cancel-button" on-tap="onCancelTap_" 27 <paper-button class="cancel-button" on-tap="onCancelTap_">
28 id="cancel">$i18n{cancel}</paper-button> 28 $i18n{cancel}
29 <paper-button id="add" on-tap="addUser_" class="action-button" disabled> 29 </paper-button>
30 <paper-button on-tap="addUser_" class="action-button"
31 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
30 $i18n{add} 32 $i18n{add}
31 </paper-button> 33 </paper-button>
32 </div> 34 </div>
33 </dialog> 35 </dialog>
34 </template> 36 </template>
35 <script src="users_add_user_dialog.js"></script> 37 <script src="users_add_user_dialog.js"></script>
36 </dom-module> 38 </dom-module>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698