Console toolbar button tooltip (Option + Command + C) doesn't work. How to reproduce: * Open the Web Inspector. * Press Option + Command + C. Expected result: The console window opens up (or closes if it was opened). Actual result: The console window doesn't open. The problem is that the shortcut tooltip is shown but its operation is not implemented.
<rdar://problem/15944643>
Created attachment 222641 [details] Patch
Comment on attachment 222641 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=222641&action=review > Source/WebInspectorUI/UserInterface/KeyboardShortcut.js:183 > + result += InspectorFrontendHost.platform() === "mac" ? "\u2325" : "\u2387"; Arn't these the same symbol?
Comment on attachment 222641 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=222641&action=review >> Source/WebInspectorUI/UserInterface/KeyboardShortcut.js:183 >> + result += InspectorFrontendHost.platform() === "mac" ? "\u2325" : "\u2387"; > > Arn't these the same symbol? I guess on Mac they might be, but not on other platforms.
Comment on attachment 222641 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=222641&action=review >>> Source/WebInspectorUI/UserInterface/KeyboardShortcut.js:183 >>> + result += InspectorFrontendHost.platform() === "mac" ? "\u2325" : "\u2387"; >> >> Arn't these the same symbol? > > I guess on Mac they might be, but not on other platforms. \u2325 is the 'Option' key symbol (http://en.wikipedia.org/wiki/Option_key) \u2387 is the 'Alternative' key symbol (http://en.wikipedia.org/wiki/Alt_key) In Mac the 'Option' key (which is also labeled as 'Alt' I think) has always the '\u2325' symbol printed on the key. In no mac keyboards, the equivalent of the 'Option' key is the 'Alt' key, and although it has no symbol printed on it I think it's better to print the correspondent Unicode symbol for the 'Alt' key. As a matter of fact, the first time I checked the shortcuts I could understand the symbols for Shift and Control, but as for the Option or Command key (Mac key) symbols I had to figure out what keys were those.
Comment on attachment 222641 [details] Patch Clearing flags on attachment: 222641 Committed r163086: <http://trac.webkit.org/changeset/163086>
All reviewed patches have been landed. Closing bug.