Opened 7 years ago
Closed 6 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)
Change History (15)
Changed 7 years ago by manuq
comment:1 Changed 7 years ago by erikos
comment:2 Changed 7 years ago by manuq
- Description modified (diff)
comment:3 Changed 7 years ago by manuq
- Cc silbe added
comment:4 Changed 7 years ago by manuq
- Milestone changed from Unspecified by Release Team to 0.96
- Version Unspecified deleted
comment:5 Changed 7 years ago by manuq
- Keywords patch added
- Milestone changed from 0.96 to 0.98
comment:6 Changed 7 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 7 years ago by humitos
- Owner changed from erikos to humitos
- Status changed from new to accepted
comment:8 Changed 7 years ago by manuq
As per erikos comment we need to show the shortcut in the buttons tooltips before.
comment:9 Changed 7 years ago by humitos
I've just uploaded a new version for this patch. Can you check it, please?
comment:10 Changed 7 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 7 years ago by manuq
- Milestone changed from 0.98 to Unspecified by Release Team
Moving out of 0.98 .
comment:12 Changed 6 years ago by humitos
- Priority changed from Normal to Low
comment:13 Changed 6 years ago by manuq
- Resolution set to wontfix
- Status changed from accepted to closed
Ok, if we care about it, we should make sure the shortcut is discoverable as well. Display it in the palette.