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

Side by Side Diff: chrome/browser/resources/options/manage_profile_overlay.html

Issue 8715009: Add profile avatar to profile delete dialog. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Removed an extra ; from the css file. Created 9 years 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 <div id="manage-profile-overlay" class="page" hidden> 1 <div id="manage-profile-overlay" class="page" hidden>
2 <!-- Dialog for managing profiles. --> 2 <!-- Dialog for managing profiles. -->
3 <div id="manage-profile-overlay-manage" hidden> 3 <div id="manage-profile-overlay-manage" hidden>
4 <h1 i18n-content="manageProfilesTitle"></h1> 4 <h1 i18n-content="manageProfilesTitle"></h1>
5 <div id="manage-profile-content" class="content-area"> 5 <div id="manage-profile-content" class="content-area">
6 <div id="manage-profile-name-div"> 6 <div id="manage-profile-name-div">
7 <span id="manage-profile-name-label" 7 <span id="manage-profile-name-label"
8 i18n-content="manageProfilesNameLabel"></span> 8 i18n-content="manageProfilesNameLabel"></span>
9 <input id="manage-profile-name" type="text" required> 9 <input id="manage-profile-name" type="text" required>
10 </div> 10 </div>
11 <div id="manage-profile-error-bubble" hidden> 11 <div id="manage-profile-error-bubble" hidden>
12 </div> 12 </div>
13 <div id="manage-profile-icon-label" 13 <div id="manage-profile-icon-label"
14 i18n-content="manageProfilesIconLabel"></div> 14 i18n-content="manageProfilesIconLabel"></div>
15 <grid id="manage-profile-icon-grid"></grid> 15 <grid id="manage-profile-icon-grid"></grid>
16 </div> 16 </div>
17 <div class="action-area"> 17 <div class="action-area">
18 <div class="button-strip"> 18 <div class="button-strip">
19 <button id="manage-profile-cancel" i18n-content="cancel"></button> 19 <button id="manage-profile-cancel" i18n-content="cancel"></button>
20 <button id="manage-profile-ok" i18n-content="ok"></button> 20 <button id="manage-profile-ok" i18n-content="ok"></button>
21 </div> 21 </div>
22 </div> 22 </div>
23 </div> 23 </div>
24 <!-- Dialog for deleting profiles. --> 24 <!-- Dialog for deleting profiles. -->
25 <div id="manage-profile-overlay-delete" hidden> 25 <div id="manage-profile-overlay-delete" hidden>
26 <h1 i18n-content="deleteProfileTitle"></h1> 26 <h1 i18n-content="deleteProfileTitle"></h1>
27 <div class="content-area"> 27 <div id="delete-profile-content" class="content-area">
28 <img id="delete-profile-avatar" class="profile-icon">
Evan Stade 2011/11/29 00:17:38 instead of adding an <img>, set the image as the b
jwd 2011/11/29 17:13:44 I would expect RTL languages to put the avatar on
Evan Stade 2011/11/30 00:28:34 yes, use a selector with html[dir=rtl]
jwd 2011/11/30 16:48:04 Done. Though it does seem to me that the image is
28 <div id="delete-profile-message"></div> 29 <div id="delete-profile-message"></div>
29 </div> 30 </div>
30 <div class="action-area"> 31 <div class="action-area">
31 <div class="button-strip"> 32 <div class="button-strip">
32 <button id="delete-profile-cancel" i18n-content="cancel"></button> 33 <button id="delete-profile-cancel" i18n-content="cancel"></button>
33 <button id="delete-profile-ok" 34 <button id="delete-profile-ok"
34 i18n-content="deleteProfileOK"></button> 35 i18n-content="deleteProfileOK"></button>
35 </div> 36 </div>
36 </div> 37 </div>
37 </div> 38 </div>
38 </div> 39 </div>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698