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

Side by Side Diff: chrome/browser/ui/views/harmony/chrome_layout_provider.h

Issue 2932523003: Fix Harmony popover buttons being flush with their content. (Closed)
Patch Set: fix merge Created 3 years, 6 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
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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_HARMONY_CHROME_LAYOUT_PROVIDER_H_ 5 #ifndef CHROME_BROWSER_UI_VIEWS_HARMONY_CHROME_LAYOUT_PROVIDER_H_
6 #define CHROME_BROWSER_UI_VIEWS_HARMONY_CHROME_LAYOUT_PROVIDER_H_ 6 #define CHROME_BROWSER_UI_VIEWS_HARMONY_CHROME_LAYOUT_PROVIDER_H_
7 7
8 #include <memory> 8 #include <memory>
9 9
10 #include "base/macros.h" 10 #include "base/macros.h"
11 #include "ui/gfx/geometry/insets.h" 11 #include "ui/gfx/geometry/insets.h"
12 #include "ui/views/layout/grid_layout.h" 12 #include "ui/views/layout/grid_layout.h"
13 #include "ui/views/layout/layout_provider.h" 13 #include "ui/views/layout/layout_provider.h"
14 14
15 enum ChromeDistanceMetric { 15 enum ChromeDistanceMetric {
16 // Default minimum width of a button. 16 // Default minimum width of a button.
17 DISTANCE_BUTTON_MINIMUM_WIDTH = views::VIEWS_DISTANCE_END, 17 DISTANCE_BUTTON_MINIMUM_WIDTH = views::VIEWS_DISTANCE_END,
18 // Vertical spacing between a list of multiple controls in one column. 18 // Vertical spacing between a list of multiple controls in one column.
19 DISTANCE_CONTROL_LIST_VERTICAL, 19 DISTANCE_CONTROL_LIST_VERTICAL,
20 // Spacing between a dialog button and the content above it.
21 DISTANCE_DIALOG_BUTTON_TOP,
22 // Smaller horizontal spacing between other controls that are logically 20 // Smaller horizontal spacing between other controls that are logically
23 // related. 21 // related.
24 DISTANCE_RELATED_CONTROL_HORIZONTAL_SMALL, 22 DISTANCE_RELATED_CONTROL_HORIZONTAL_SMALL,
25 // Smaller vertical spacing between controls that are logically related. 23 // Smaller vertical spacing between controls that are logically related.
26 DISTANCE_RELATED_CONTROL_VERTICAL_SMALL, 24 DISTANCE_RELATED_CONTROL_VERTICAL_SMALL,
27 // Horizontal spacing between an item such as an icon or checkbox and a 25 // Horizontal spacing between an item such as an icon or checkbox and a
28 // label related to it. 26 // label related to it.
29 DISTANCE_RELATED_LABEL_HORIZONTAL, 27 DISTANCE_RELATED_LABEL_HORIZONTAL,
30 // Horizontal indent of a subsection relative to related items above, e.g. 28 // Horizontal indent of a subsection relative to related items above, e.g.
31 // checkboxes below explanatory text/headings. 29 // checkboxes below explanatory text/headings.
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
72 // types, e.g. by adding a new LayoutDistance value that means what you need. 70 // types, e.g. by adding a new LayoutDistance value that means what you need.
73 // 71 //
74 // TODO(pkasting): Fix callers and remove this. 72 // TODO(pkasting): Fix callers and remove this.
75 virtual bool IsHarmonyMode() const; 73 virtual bool IsHarmonyMode() const;
76 74
77 private: 75 private:
78 DISALLOW_COPY_AND_ASSIGN(ChromeLayoutProvider); 76 DISALLOW_COPY_AND_ASSIGN(ChromeLayoutProvider);
79 }; 77 };
80 78
81 #endif // CHROME_BROWSER_UI_VIEWS_HARMONY_CHROME_LAYOUT_PROVIDER_H_ 79 #endif // CHROME_BROWSER_UI_VIEWS_HARMONY_CHROME_LAYOUT_PROVIDER_H_
OLDNEW
« no previous file with comments | « chrome/browser/ui/views/bookmarks/bookmark_bubble_view.cc ('k') | chrome/browser/ui/views/harmony/chrome_layout_provider.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698