diff options
author | ben@chromium.org <ben@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-08 00:34:05 +0000 |
---|---|---|
committer | ben@chromium.org <ben@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-08 00:34:05 +0000 |
commit | 2362e4fe2905ab75d3230ebc3e307ae53e2b8362 (patch) | |
tree | e6d88357a2021811e0e354f618247217be8bb3da /views/accelerator.cc | |
parent | db23ac3e713dc17509b2b15d3ee634968da45715 (diff) | |
download | chromium_src-2362e4fe2905ab75d3230ebc3e307ae53e2b8362.zip chromium_src-2362e4fe2905ab75d3230ebc3e307ae53e2b8362.tar.gz chromium_src-2362e4fe2905ab75d3230ebc3e307ae53e2b8362.tar.bz2 |
Move src/chrome/views to src/views. RS=darin http://crbug.com/11387
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@15604 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'views/accelerator.cc')
-rw-r--r-- | views/accelerator.cc | 130 |
1 files changed, 130 insertions, 0 deletions
diff --git a/views/accelerator.cc b/views/accelerator.cc new file mode 100644 index 0000000..b9a474a --- /dev/null +++ b/views/accelerator.cc @@ -0,0 +1,130 @@ +// Copyright (c) 2006-2008 The Chromium Authors. All rights reserved. +// Use of this source code is governed by a BSD-style license that can be +// found in the LICENSE file. + +#include "views/accelerator.h" + +#include <windows.h> + +#include "app/l10n_util.h" +#include "base/logging.h" +#include "base/string_util.h" +#include "grit/generated_resources.h" + +namespace views { + +std::wstring Accelerator::GetShortcutText() const { + int string_id = 0; + switch(key_code_) { + case VK_TAB: + string_id = IDS_TAB_KEY; + break; + case VK_RETURN: + string_id = IDS_ENTER_KEY; + break; + case VK_ESCAPE: + string_id = IDS_ESC_KEY; + break; + case VK_PRIOR: + string_id = IDS_PAGEUP_KEY; + break; + case VK_NEXT: + string_id = IDS_PAGEDOWN_KEY; + break; + case VK_END: + string_id = IDS_END_KEY; + break; + case VK_HOME: + string_id = IDS_HOME_KEY; + break; + case VK_INSERT: + string_id = IDS_INSERT_KEY; + break; + case VK_DELETE: + string_id = IDS_DELETE_KEY; + break; + case VK_F1: + string_id = IDS_F1_KEY; + break; + case VK_F11: + string_id = IDS_F11_KEY; + break; + } + + std::wstring shortcut; + if (!string_id) { + // Our fallback is to try translate the key code to a regular character + // unless it is one of digits (VK_0 to VK_9). Some keyboard + // layouts have characters other than digits assigned in + // an unshifted mode (e.g. French AZERY layout has 'a with grave + // accent' for '0'). For display in the menu (e.g. Ctrl-0 for the + // default zoom level), we leave VK_[0-9] alone without translation. + wchar_t key; + if (key_code_ >= '0' && key_code_ <= '9') + key = key_code_; + else + key = LOWORD(::MapVirtualKeyW(key_code_, MAPVK_VK_TO_CHAR)); + shortcut += key; + } else { + shortcut = l10n_util::GetString(string_id); + } + + // Checking whether the character used for the accelerator is alphanumeric. + // If it is not, then we need to adjust the string later on if the locale is + // right-to-left. See below for more information of why such adjustment is + // required. + std::wstring shortcut_rtl; + bool adjust_shortcut_for_rtl = false; + if (l10n_util::GetTextDirection() == l10n_util::RIGHT_TO_LEFT && + shortcut.length() == 1 && + !IsAsciiAlpha(shortcut.at(0)) && + !IsAsciiDigit(shortcut.at(0))) { + adjust_shortcut_for_rtl = true; + shortcut_rtl.assign(shortcut); + } + + if (IsShiftDown()) + shortcut = l10n_util::GetStringF(IDS_SHIFT_MODIFIER, shortcut); + + // Note that we use 'else-if' in order to avoid using Ctrl+Alt as a shortcut. + // See http://blogs.msdn.com/oldnewthing/archive/2004/03/29/101121.aspx for + // more information. + if (IsCtrlDown()) + shortcut = l10n_util::GetStringF(IDS_CONTROL_MODIFIER, shortcut); + else if (IsAltDown()) + shortcut = l10n_util::GetStringF(IDS_ALT_MODIFIER, shortcut); + + // For some reason, menus in Windows ignore standard Unicode directionality + // marks (such as LRE, PDF, etc.). On RTL locales, we use RTL menus and + // therefore any text we draw for the menu items is drawn in an RTL context. + // Thus, the text "Ctrl++" (which we currently use for the Zoom In option) + // appears as "++Ctrl" in RTL because the Unicode BiDi algorithm puts + // punctuations on the left when the context is right-to-left. Shortcuts that + // do not end with a punctuation mark (such as "Ctrl+H" do not have this + // problem). + // + // The only way to solve this problem is to adjust the string if the locale + // is RTL so that it is drawn correnctly in an RTL context. Instead of + // returning "Ctrl++" in the above example, we return "++Ctrl". This will + // cause the text to appear as "Ctrl++" when Windows draws the string in an + // RTL context because the punctunation no longer appears at the end of the + // string. + // + // TODO(idana) bug# 1232732: this hack can be avoided if instead of using + // views::Menu we use views::MenuItemView because the latter is a View + // subclass and therefore it supports marking text as RTL or LTR using + // standard Unicode directionality marks. + if (adjust_shortcut_for_rtl) { + int key_length = static_cast<int>(shortcut_rtl.length()); + DCHECK_GT(key_length, 0); + shortcut_rtl.append(L"+"); + + // Subtracting the size of the shortcut key and 1 for the '+' sign. + shortcut_rtl.append(shortcut, 0, shortcut.length() - key_length - 1); + shortcut.swap(shortcut_rtl); + } + + return shortcut; +} + +} // namespace views |