diff options
author | nkostylev@chromium.org <nkostylev@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-10-29 09:23:23 +0000 |
---|---|---|
committer | nkostylev@chromium.org <nkostylev@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-10-29 09:23:23 +0000 |
commit | 9af68c48d58e3eb092d21065878aaf0ac63b3934 (patch) | |
tree | 641b0cb020b7c73d4beb58db9817d7cfe3c17dee | |
parent | 314d280dc175363aa4cfe5da28592b6f2e1c45c3 (diff) | |
download | chromium_src-9af68c48d58e3eb092d21065878aaf0ac63b3934.zip chromium_src-9af68c48d58e3eb092d21065878aaf0ac63b3934.tar.gz chromium_src-9af68c48d58e3eb092d21065878aaf0ac63b3934.tar.bz2 |
[cros] Welcome screen fixes.
- Text in combo boxes is blue, should be black
- Increase welcome title font size
- Increase combobox control height a bit
BUG=chromium-os:8305
TEST=manual
Review URL: http://codereview.chromium.org/4140004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64390 0039d316-1c4b-4281-b951-d872f2087c98
-rw-r--r-- | chrome/browser/chromeos/login/network_selection_view.cc | 42 |
1 files changed, 22 insertions, 20 deletions
diff --git a/chrome/browser/chromeos/login/network_selection_view.cc b/chrome/browser/chromeos/login/network_selection_view.cc index e24e205..20e87fd 100644 --- a/chrome/browser/chromeos/login/network_selection_view.cc +++ b/chrome/browser/chromeos/login/network_selection_view.cc @@ -60,8 +60,11 @@ const int kPaddingColumnWidth = 55; const int kMediumPaddingColumnWidth = 20; const int kControlPaddingRow = 15; +// Size to add to the welcome title font. +const int kWelcomeTitleFontDelta = 5; + // Fixed size for language/keyboard/network controls height. -const int kSelectionBoxHeight = 29; +const int kSelectionBoxHeight = 30; // Menu button is drawn using our custom icons in resources. See // TextButtonBorder::Paint() for details. So this offset compensate @@ -83,6 +86,19 @@ static const int kProxySettingsDialogReasonableWidth = 750; static const int kProxySettingsDialogReasonableHeight = 460; static const int kProxySettingsDialogReasonableWidthRatio = 0.4; static const int kProxySettingsDialogReasonableHeightRatio = 0.4; + +// Initializes menu button default properties. +static void InitMenuButtonProperties(views::MenuButton* menu_button) { + menu_button->SetFocusable(true); + menu_button->SetNormalHasBorder(true); + menu_button->SetEnabledColor(SK_ColorBLACK); + menu_button->SetHighlightColor(SK_ColorBLACK); + menu_button->SetHoverColor(SK_ColorBLACK); + menu_button->set_animate_on_state_change(false); + // Menu is positioned by bottom right corner of the MenuButton. + menu_button->set_menu_offset(kMenuHorizontalOffset, kMenuVerticalOffset); +} + } // namespace namespace chromeos { @@ -276,8 +292,8 @@ void NetworkSelectionView::Init() { views::Background::CreateBackgroundPainter(true, painter)); ResourceBundle& rb = ResourceBundle::GetSharedInstance(); - gfx::Font welcome_label_font = - rb.GetFont(ResourceBundle::LargeFont).DeriveFont(0, gfx::Font::BOLD); + gfx::Font welcome_label_font = rb.GetFont(ResourceBundle::LargeFont). + DeriveFont(kWelcomeTitleFontDelta, gfx::Font::BOLD); welcome_label_ = new views::Label(); welcome_label_->SetColor(kWelcomeColor); @@ -289,12 +305,7 @@ void NetworkSelectionView::Init() { languages_menubutton_ = new NotifyingMenuButton( NULL, std::wstring(), delegate_->language_switch_menu(), true, delegate_); - languages_menubutton_->SetFocusable(true); - languages_menubutton_->SetNormalHasBorder(true); - languages_menubutton_->set_animate_on_state_change(false); - // Menu is positioned by bottom right corner of the MenuButton. - languages_menubutton_->set_menu_offset(kMenuHorizontalOffset, - kMenuVerticalOffset); + InitMenuButtonProperties(languages_menubutton_); select_keyboard_label_ = new views::Label(); select_keyboard_label_->SetFont(rb.GetFont(ResourceBundle::MediumFont)); @@ -302,11 +313,7 @@ void NetworkSelectionView::Init() { keyboards_menubutton_ = new views::MenuButton( NULL /* listener */, L"", delegate_->keyboard_switch_menu(), true /* show_menu_marker */); - keyboards_menubutton_->SetFocusable(true); - keyboards_menubutton_->SetNormalHasBorder(true); - keyboards_menubutton_->set_animate_on_state_change(false); - keyboards_menubutton_->set_menu_offset(kMenuHorizontalOffset, - kMenuVerticalOffset); + InitMenuButtonProperties(keyboards_menubutton_); select_network_label_ = new views::Label(); select_network_label_->SetFont(rb.GetFont(ResourceBundle::MediumFont)); @@ -314,12 +321,7 @@ void NetworkSelectionView::Init() { network_dropdown_ = new NetworkControlReportOnActivate(false, GetNativeWindow(), delegate_); - - network_dropdown_->set_menu_offset(kMenuHorizontalOffset, - kMenuVerticalOffset); - network_dropdown_->SetNormalHasBorder(true); - network_dropdown_->SetFocusable(true); - network_dropdown_->set_animate_on_state_change(false); + InitMenuButtonProperties(network_dropdown_); connecting_network_label_ = new views::Label(); connecting_network_label_->SetFont(rb.GetFont(ResourceBundle::MediumFont)); |