Opened 11 years ago
Closed 10 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 11 years ago by manuq
comment:1 Changed 11 years ago by erikos
comment:2 Changed 11 years ago by manuq
- Description modified (diff)
comment:3 Changed 11 years ago by manuq
- Cc silbe added
comment:4 Changed 11 years ago by manuq
- Milestone changed from Unspecified by Release Team to 0.96
- Version Unspecified deleted
comment:5 Changed 11 years ago by manuq
- Keywords patch added
- Milestone changed from 0.96 to 0.98
comment:6 Changed 11 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 11 years ago by humitos
- Owner changed from erikos to humitos
- Status changed from new to accepted
comment:8 Changed 11 years ago by manuq
As per erikos comment we need to show the shortcut in the buttons tooltips before.
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 10 years ago by humitos
- Priority changed from Normal to Low
comment:13 Changed 10 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.