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

Unified Diff: chrome/browser/ui/constrained_window.h

Issue 11077006: Correct padding and focus ring for frameless constrained window dialog (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Fix mac build, part 2 Created 8 years, 2 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/ui/constrained_window.h
diff --git a/chrome/browser/ui/constrained_window.h b/chrome/browser/ui/constrained_window.h
index 21efd6512df764d60d64ccba88fbcdcaf2e02e3d..bd5e99532a9492531bf421cc9e08409c20d9d3a5 100644
--- a/chrome/browser/ui/constrained_window.h
+++ b/chrome/browser/ui/constrained_window.h
@@ -18,10 +18,16 @@
//
class ConstrainedWindow {
public:
- static const int kVerticalPadding = 14; // top/bottom padding.
- static const int kHorizontalPadding = 17; // left/right padding.
- static const int kRowPadding = 20; // Vertical margin between dialog rows.
- static const int kBorderRadius = 2; // Border radius for dialog corners.
+ static const int kTitleTopPadding = 15; // Padding above the title.
+ static const int kHorizontalPadding = 20; // Left and right padding.
+ static const int kClientTopPadding = 18; // Padding above the client view.
+ static const int kClientBottomPadding = 20; // Padding below the client view.
+ static const int kCloseButtonPadding = 7; // Padding around the close button.
+ static const int kBorderRadius = 2; // Border radius for dialog corners.
+
+#if defined(OS_MACOSX)
+ static const int kRowPadding = 20; // Padding between rows of text.
+#endif
// Font style for dialog text.
static const ui::ResourceBundle::FontStyle kTextFontStyle =

Powered by Google App Engine
This is Rietveld 408576698