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

Unified Diff: chrome/browser/ui/search/search_ui.h

Issue 11428149: Alternate NTP: Move bookmark bar to bottom (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: add tests Created 8 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: chrome/browser/ui/search/search_ui.h
diff --git a/chrome/browser/ui/search/search_ui.h b/chrome/browser/ui/search/search_ui.h
index 2b5ceedf87e347c9c66fa8258957c88cde786f73..d063fdefe9fbf66d4990499efb1081f5fe634f19 100644
--- a/chrome/browser/ui/search/search_ui.h
+++ b/chrome/browser/ui/search/search_ui.h
@@ -8,10 +8,19 @@
namespace chrome {
namespace search {
-// The mininum height of content view to layout detached bookmark bar at bottom
-// for |NTP| search mode, calculated from chrome/browser/resources/ntp_search/
-// tile_page.js HEIGHT_FOR_BOTTOM_PANEL constant.
-extern const int kMinContentHeightForBottomBookmarkBar;
+enum {
+ // The minimum height of the content view for which the detached bookmark bar
+ // should be visible. This value is calculated from the
+ // chrome/browser/resources/ntp_search/ tile_page.js
+ // HEIGHT_FOR_BOTTOM_PANEL constant.
+ kMinContentHeightForBottomBookmarkBar = 558,
dhollowa 2012/12/04 01:24:49 I'm not a fan of using enums as integer constants.
sail 2012/12/04 01:41:12 Done.
+
+ // The maximum width of the detached bookmark bar.
+ kMaxWidthForBottomBookmarkBar = 720,
+
+ // The left and right padding of the detached bookmark bar.
+ kHorizontalPaddingForBottomBookmarkBar = 130,
+};
} // namespace search
} // namespace chrome

Powered by Google App Engine
This is Rietveld 408576698