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

Side by Side Diff: chrome/browser/ui/panels/panel_constants.h

Issue 2157963002: Improve grammar in some comments. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: maxiumum Created 4 years, 5 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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_PANELS_PANEL_CONSTANTS_H_ 5 #ifndef CHROME_BROWSER_UI_PANELS_PANEL_CONSTANTS_H_
6 #define CHROME_BROWSER_UI_PANELS_PANEL_CONSTANTS_H_ 6 #define CHROME_BROWSER_UI_PANELS_PANEL_CONSTANTS_H_
7 7
8 #include "build/build_config.h" 8 #include "build/build_config.h"
9 9
10 namespace panel { 10 namespace panel {
11 11
12 // The height in pixels of the titlebar. 12 // The height in pixels of the titlebar.
13 static const int kTitlebarHeight = 36; 13 static const int kTitlebarHeight = 36;
14 14
15 // Absolute minimum width and height for panels, including non-client area. 15 // Absolute minimum width and height for panels, including non-client area.
16 // Should only be big enough to accomodate a close button on the reasonably 16 // Should only be big enough to accomodate a close button on the reasonably
17 // recognisable titlebar. 17 // recognisable titlebar.
18 // These numbers are semi-arbitrary. 18 // These numbers are semi-arbitrary.
19 // Motivation for 'width' is to make main buttons on the titlebar functional. 19 // Motivation for 'width' is to make main buttons on the titlebar functional.
20 // Motivation for height is to allow autosized tightly-wrapped panel with a 20 // Motivation for height is to allow autosized tightly-wrapped panel with a
21 // single line of text - so the height is set to be likely less then a titlebar, 21 // single line of text - so the height is set to be likely less than a titlebar,
22 // to make sure even small content is tightly wrapped. 22 // to make sure even small content is tightly wrapped.
23 const int kPanelMinWidth = 80; 23 const int kPanelMinWidth = 80;
24 const int kPanelMinHeight = kTitlebarHeight + 10; 24 const int kPanelMinHeight = kTitlebarHeight + 10;
25 25
26 // The panel can be minimized to 4-pixel lines. 26 // The panel can be minimized to 4-pixel lines.
27 static const int kMinimizedPanelHeight = 4; 27 static const int kMinimizedPanelHeight = 4;
28 28
29 // The size (width or height) of the app icon (taskbar icon). 29 // The size (width or height) of the app icon (taskbar icon).
30 static const int kPanelAppIconSize = 32; 30 static const int kPanelAppIconSize = 32;
31 31
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
91 enum CornerStyle { 91 enum CornerStyle {
92 NOT_ROUNDED = 0, 92 NOT_ROUNDED = 0,
93 TOP_ROUNDED = 0x1, 93 TOP_ROUNDED = 0x1,
94 BOTTOM_ROUNDED = 0x2, 94 BOTTOM_ROUNDED = 0x2,
95 ALL_ROUNDED = TOP_ROUNDED | BOTTOM_ROUNDED 95 ALL_ROUNDED = TOP_ROUNDED | BOTTOM_ROUNDED
96 }; 96 };
97 97
98 } // namespace panel 98 } // namespace panel
99 99
100 #endif // CHROME_BROWSER_UI_PANELS_PANEL_CONSTANTS_H_ 100 #endif // CHROME_BROWSER_UI_PANELS_PANEL_CONSTANTS_H_
OLDNEW
« no previous file with comments | « chrome/browser/ui/cocoa/panels/panel_window_controller_cocoa.h ('k') | chrome/browser/ui/views/frame/browser_frame.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698