Status

Firefox for Android Graveyard
General
VERIFIED WONTFIX
10 years ago
9 years ago

People

(Reporter: db48x, Assigned: db48x)

Tracking

(Blocks: 1 bug, {uiwanted})

Trunk
fennec1.0b1
x86
Linux
uiwanted
Bug Flags:
wanted-fennec1.0 -

Details

(URL)

Attachments

(1 attachment)

(Assignee)

Description

10 years ago
Currently the shortcut editor uses a tree, which makes implementing inline editing fairly easy. However, I don't think it will work well with commands that have more than one keyboard shortcut (of which there are already several.)

I'm not quite sure what will work best, though. Perhaps something similar to the prefs window, where it shows a two-line description and the primary shortcut for each command. Then, when a command is selected we can expand it vertically to include all shortcuts assigned to that command, and a button for adding new shortcuts. It may also help to group the shortcuts into categories, to make it easier to navigate the list.

Updated

10 years ago
Flags: wanted-fennec1.0+
Target Milestone: --- → Fennec A3
(Assignee)

Updated

9 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 1

9 years ago
Created attachment 356720 [details] [diff] [review]
457648-1.diff

there are a few minor polish-level things to clean up (such as what happens when you remove a key from shortcut then select a different shortcut) but I'd like to get started on the reviews.
Attachment #356720 - Flags: review?(mark.finkle)
Blocks: 477628

Updated

9 years ago
Flags: wanted-fennec1.0+
Flags: wanted-fennec1.0?
Madhava and I talked a while back about the Shortcut Editor. We came to the conclusion that it would not need to be shipped in Fennec, but there could be add-ons to either support a full scale editor or just add-ons to switch hardkeys around for specific phone models.

So I am suggesting we close this bug
closing for now
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → WONTFIX
Attachment #356720 - Flags: review?(mark.finkle)
Flags: wanted-fennec1.0? → wanted-fennec1.0-
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.