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

Unified 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, 1 month 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/options/manage_profile_overlay.html
diff --git a/chrome/browser/resources/options/manage_profile_overlay.html b/chrome/browser/resources/options/manage_profile_overlay.html
index afc48019e96ca505b7252d9376f9dc2ea133eb05..17a3982d1ff66909b5f106d8ddc7145b73eed672 100644
--- a/chrome/browser/resources/options/manage_profile_overlay.html
+++ b/chrome/browser/resources/options/manage_profile_overlay.html
@@ -24,7 +24,8 @@
<!-- Dialog for deleting profiles. -->
<div id="manage-profile-overlay-delete" hidden>
<h1 i18n-content="deleteProfileTitle"></h1>
- <div class="content-area">
+ <div id="delete-profile-content" class="content-area">
+ <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
<div id="delete-profile-message"></div>
</div>
<div class="action-area">

Powered by Google App Engine
This is Rietveld 408576698