Opened 9 years ago

Closed 8 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 9 years ago.
0001-Change-back-forward-navigation-shortcut-to-Alt-modif.2.patch​ (2.6 KB) - added by humitos 8 years ago.
v2 - show Alt+Left and Alt+Right in the tooltip as accelerator

Download all attachments as: .zip

Change History (15)

Changed 9 years ago by manuq

comment:1 Changed 9 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 9 years ago by manuq

  • Description modified (diff)

comment:3 Changed 9 years ago by manuq

  • Cc silbe added

comment:4 Changed 9 years ago by manuq

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

comment:5 Changed 9 years ago by manuq

  • Keywords patch added
  • Milestone changed from 0.96 to 0.98

comment:6 Changed 8 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 8 years ago by humitos

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

comment:8 Changed 8 years ago by manuq

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

Changed 8 years ago by humitos

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

comment:9 Changed 8 years ago by humitos

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

comment:10 Changed 8 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 8 years ago by manuq

  • Milestone changed from 0.98 to Unspecified by Release Team

Moving out of 0.98 .

comment:12 Changed 8 years ago by humitos

  • Priority changed from Normal to Low

comment:13 Changed 8 years ago by manuq

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