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

Issue 325004: Make user shortcuts override built-in shortcuts. (Closed)

Created:
11 years, 2 months ago by Nico
Modified:
9 years, 7 months ago
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Make user shortcuts override built-in shortcuts. BUG=25330 TEST=Open sysprefs, keyboard, keyboard shortcuts, add a keyboard shortcut that makes cmd-right do "Select Next Tab". Hit cmd-right in chromium. Should switch tabs instead of doing history navigation (note "instead". If it does both tab switching _and_ history navigation, that's wrong). If a text box is focussed, cmd-right should however instead move the cursor and not switch tabs. Switching windows with cmd-` should still work. Switching tabs with cmd-1/2/3 and cmd-{/} should still work. Using backspace/shift-backspace for history should still work. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=29877

Patch Set 1 #

Total comments: 2

Patch Set 2 : comment #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -0 lines) Patch
M chrome/browser/tab_contents/tab_contents_view_mac.mm View 1 1 chunk +8 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Nico
11 years, 2 months ago (2009-10-22 22:54:03 UTC) #1
jeremy
LGTM http://codereview.chromium.org/325004/diff/1/2 File chrome/browser/tab_contents/tab_contents_view_mac.mm (right): http://codereview.chromium.org/325004/diff/1/2#newcode340 Line 340: // dispatching to the menu as well. ...
11 years, 2 months ago (2009-10-23 05:22:54 UTC) #2
Nico
11 years, 2 months ago (2009-10-23 06:08:12 UTC) #3
Thanks, submitting.

http://codereview.chromium.org/325004/diff/1/2
File chrome/browser/tab_contents/tab_contents_view_mac.mm (right):

http://codereview.chromium.org/325004/diff/1/2#newcode340
Line 340: // dispatching to the menu as well.
On 2009/10/23 05:22:54, jeremy wrote:
> "dispatching the menu" -> "Invoking the original menu item shortcut" ?

Done.

Powered by Google App Engine
This is Rietveld 408576698