Opened 12 years ago

Closed 11 years ago

#3536 closed enhancement (wontfix)

Browse: change back/forward navigation shortcut to Alt modifier

Reported by: manuq Owned by: humitos
Priority: Low Milestone: Unspecified
Component: Browse Version:
Severity: Unspecified Keywords: patch
Cc: silbe, manuq, humitos Distribution/OS: Unspecified
Bug Status: Unconfirmed

Description (last modified by manuq)

Alt is used in most browsers (Firefox, Epiphany) for back/forward navigation modifier and the left/right cursor keys. That's nice because Ctrl modifier and left/right keys is commonly used when editing text to move the cursor to previous/next word. I find frustrating the current behaviour, and catched me while editing a web form (like this I'm typing now in trac) or modifying the URL entry.

I wonder if is there a good reason to use Ctrl. Maybe because all other acceleratos use this modifier? This change was introduced in commit 972e93444546ff9320ab7ebfd9919696dc575887 by Sascha Silbe.

The patch attached changes the modifier to Alt.

Attachments (2)

0001-Change-back-forward-navigation-shortcut-to-Alt-modif.patch (1.7 KB) - added by manuq 12 years ago.
0001-Change-back-forward-navigation-shortcut-to-Alt-modif.2.patch (2.6 KB) - added by humitos 11 years ago.
v2 - show Alt+Left and Alt+Right in the tooltip as accelerator

Download all attachments as: .zip

Change History (15)

comment:1 Changed 12 years ago by erikos

Ok, if we care about it, we should make sure the shortcut is discoverable as well. Display it in the palette.

comment:2 Changed 12 years ago by manuq

  • Description modified (diff)

comment:3 Changed 12 years ago by manuq

  • Cc silbe added

comment:4 Changed 12 years ago by manuq

  • Milestone changed from Unspecified by Release Team to 0.96
  • Version Unspecified deleted

comment:5 Changed 12 years ago by manuq

  • Keywords patch added
  • Milestone changed from 0.96 to 0.98

comment:6 Changed 12 years ago by humitos

  • Cc manuq humitos added
  • Priority changed from Unspecified by Maintainer to Normal

gonzalo or manuq, can you take a look at and apply this patch?

comment:7 Changed 12 years ago by humitos

  • Owner changed from erikos to humitos
  • Status changed from new to accepted

comment:8 Changed 12 years ago by manuq

As per erikos comment we need to show the shortcut in the buttons tooltips before.

Changed 11 years ago by humitos

v2 - show Alt+Left and Alt+Right in the tooltip as accelerator

comment:9 Changed 11 years ago by humitos

I've just uploaded a new version for this patch. Can you check it, please?

comment:10 Changed 11 years ago by manuq

Thanks for your work humitos. And sorry, now I think we need a proper discussion for such a change, and this is not priority for this cycle.

comment:11 Changed 11 years ago by manuq

  • Milestone changed from 0.98 to Unspecified by Release Team

Moving out of 0.98 .

comment:12 Changed 11 years ago by humitos

  • Priority changed from Normal to Low

comment:13 Changed 11 years ago by manuq

  • Resolution set to wontfix
  • Status changed from accepted to closed
Note: See TracTickets for help on using tickets.