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

Unified Diff: ui/message_center/message_center_style.h

Issue 102473005: Refresh for the Chrome notifications image template. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 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 side-by-side diff with in-line comments
Download patch
Index: ui/message_center/message_center_style.h
diff --git a/ui/message_center/message_center_style.h b/ui/message_center/message_center_style.h
index 7b7f1aa81e215767e39a6c9513fe30ecb6071418..0efdf68dff2edf758ff303c51959e96b5314a9f6 100644
--- a/ui/message_center/message_center_style.h
+++ b/ui/message_center/message_center_style.h
@@ -17,8 +17,10 @@ namespace message_center {
// Square image sizes in DIPs.
const int kNotificationButtonIconSize = 16;
const int kNotificationIconSize = 80;
+// A border is applied to images that have a non-preferred aspect ratio.
+const int kNotificationImageBorderSize = 10;
const int kNotificationPreferredImageSize = 360;
-const float kNotificationPreferredImageRatio = 1.5;
+const float kNotificationPreferredImageRatio = 0.6666667f;
Dmitry Titov 2013/12/06 19:29:11 Why not just 0.67f with a comment explaining the o
dewittj 2013/12/10 02:47:04 Done.
const int kSettingsIconSize = 16;
// Limits.
@@ -66,7 +68,8 @@ const int kMessageLineHeight = 18; // In DIPs.
// Colors.
extern const SkColor kNotificationBackgroundColor; // Background of the card.
-extern const SkColor kIconBackgroundColor; // Used behind icons smaller.
+extern const SkColor kImageBackgroundColor; // Background of the image.
+extern const SkColor kIconBackgroundColor; // Used behind icons smaller
// than the icon view.
extern const SkColor kRegularTextColor; // Title, message, ...
extern const SkColor kDimTextColor;

Powered by Google App Engine
This is Rietveld 408576698