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

Side by Side Diff: chrome/browser/ui/views/layout_constants.h

Issue 1399253005: Fix layout of omnibox rows for material design (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: comments addressed Created 5 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 unified diff | Download patch
« no previous file with comments | « no previous file | chrome/browser/ui/views/layout_constants.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_UI_VIEWS_LAYOUT_CONSTANTS_H_ 5 #ifndef CHROME_BROWSER_UI_VIEWS_LAYOUT_CONSTANTS_H_
6 #define CHROME_BROWSER_UI_VIEWS_LAYOUT_CONSTANTS_H_ 6 #define CHROME_BROWSER_UI_VIEWS_LAYOUT_CONSTANTS_H_
7 7
8 #include "ui/gfx/geometry/insets.h" 8 #include "ui/gfx/geometry/insets.h"
9 9
10 enum LayoutConstant { 10 enum LayoutConstant {
(...skipping 72 matching lines...) Expand 10 before | Expand all | Expand 10 after
83 83
84 // The horizontal space between most items in the toolbar. 84 // The horizontal space between most items in the toolbar.
85 TOOLBAR_STANDARD_SPACING, 85 TOOLBAR_STANDARD_SPACING,
86 }; 86 };
87 87
88 enum LayoutInset { 88 enum LayoutInset {
89 // The padding between the avatar icon and the frame border on the left, the 89 // The padding between the avatar icon and the frame border on the left, the
90 // tabstrip on the right, and the toolbar on the bottom. 90 // tabstrip on the right, and the toolbar on the bottom.
91 AVATAR_ICON, 91 AVATAR_ICON,
92 92
93 // The padding above the top row and below the bottom row in the omnibox
94 // dropdown.
95 OMNIBOX_DROPDOWN,
96
93 // In an omnibox dropdown row, the minimum distance between the icon and the 97 // In an omnibox dropdown row, the minimum distance between the icon and the
94 // row edge. 98 // row edge.
95 OMNIBOX_DROPDOWN_ICON, 99 OMNIBOX_DROPDOWN_ICON,
96 100
97 // In an omnibox dropdown row, the minimum distance between the text and the 101 // In an omnibox dropdown row, the minimum distance between the text and the
98 // row edge. 102 // row edge.
99 OMNIBOX_DROPDOWN_TEXT, 103 OMNIBOX_DROPDOWN_TEXT,
100 104
101 // The padding inside the tab bounds that defines the tab contents region. 105 // The padding inside the tab bounds that defines the tab contents region.
102 TAB, 106 TAB,
103 107
104 // The minimum padding of the toolbar. The edge graphics have some built-in 108 // The minimum padding of the toolbar. The edge graphics have some built-in
105 // spacing, shadowing, so this accounts for that as well. 109 // spacing, shadowing, so this accounts for that as well.
106 TOOLBAR, 110 TOOLBAR,
107 111
108 // The spacing between a ToolbarButton's image and its border. 112 // The spacing between a ToolbarButton's image and its border.
109 TOOLBAR_BUTTON, 113 TOOLBAR_BUTTON,
110 }; 114 };
111 115
112 int GetLayoutConstant(LayoutConstant constant); 116 int GetLayoutConstant(LayoutConstant constant);
113 gfx::Insets GetLayoutInsets(LayoutInset inset); 117 gfx::Insets GetLayoutInsets(LayoutInset inset);
114 118
115 #endif // CHROME_BROWSER_UI_VIEWS_LAYOUT_CONSTANTS_H_ 119 #endif // CHROME_BROWSER_UI_VIEWS_LAYOUT_CONSTANTS_H_
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/ui/views/layout_constants.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698