Chromium Code Reviews| Index: chrome/browser/ui/views/harmony/layout_delegate.h |
| diff --git a/chrome/browser/ui/views/harmony/layout_delegate.h b/chrome/browser/ui/views/harmony/layout_delegate.h |
| index 6213d1c90c0484e3b5df65635f49cb379dd6b816..e1b1a86c80b9c107be67333841c319ea2d6a6163 100644 |
| --- a/chrome/browser/ui/views/harmony/layout_delegate.h |
| +++ b/chrome/browser/ui/views/harmony/layout_delegate.h |
| @@ -10,12 +10,14 @@ |
| class LayoutDelegate { |
| public: |
| enum class Metric { |
| + // Padding on the left and right side of a button's label. |
| + BUTTON_HORIZONTAL_PADDING, |
| // Margin between the edge of a dialog and the left, right, or bottom of a |
| // contained button. |
| DIALOG_BUTTON_MARGIN, |
| - // In theory, this is the spacing between a dialog button and the content |
| - // above it. In practice, what the code does with this value, at least |
| - // pre-Harmony, defies easy explanation. |
| + // Minimum width of a dialog button. |
| + DIALOG_BUTTON_MINIMUM_WIDTH, |
| + // This is the spacing between a dialog button and the content above it. |
|
Peter Kasting
2017/02/16 01:14:47
Nit: Remove "This is the"
Bret
2017/02/16 01:31:04
Done.
|
| DIALOG_BUTTON_TOP_SPACING, |
| // Horizontal or vertical margin between the edge of a dialog and the close |
| // button in the upper trailing corner. |
| @@ -36,6 +38,10 @@ class LayoutDelegate { |
| // Horizontal indent of a subsection relative to related items above, e.g. |
| // checkboxes below explanatory text/headings. |
| SUBSECTION_HORIZONTAL_INDENT, |
| + // Horizontal spacing between controls that are logically unrelated. |
| + UNRELATED_CONTROL_HORIZONTAL_SPACING, |
| + // Larger horizontal spacing between unrelated controls. |
| + UNRELATED_CONTROL_HORIZONTAL_SPACING_LARGE, |
| // Vertical spacing between controls that are logically unrelated. |
| UNRELATED_CONTROL_VERTICAL_SPACING, |
| // Larger vertical spacing between unrelated controls. |