Add "Keyboard Shortcuts" item to the Help menu

RESOLVED FIXED in Firefox 25

Status

()

Firefox
Menus
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: dao, Assigned: dao)

Tracking

(Blocks: 1 bug)

Trunk
Firefox 25
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
Created attachment 779654 [details] [diff] [review]
patch

We can just link to SUMO until bug 492557 is done. It's wanted for bugs such as bug 513165, bug 513168, bug 519937.
Attachment #779654 - Flags: review?(mdeboer)
Comment on attachment 779654 [details] [diff] [review]
patch

Review of attachment 779654 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM! I didn't know we had 'pressing' dependencies on bug 492557, but this is a good intermediate step.
Attachment #779654 - Flags: review?(mdeboer) → review+
(Assignee)

Comment 2

4 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/2c5f842a102c
(Assignee)

Updated

4 years ago
Depends on: 519927
No ui-review? Was this discussed more widely than just in this bug?
(Assignee)

Comment 4

4 years ago
(In reply to :Gavin Sharp (use gavin@gavinsharp.com for email) from comment #3)
> No ui-review? Was this discussed more widely than just in this bug?

In bug 513168, bug 519927 and of course bug 492557 that was going to add the same menu item just with a different command.
https://hg.mozilla.org/mozilla-central/rev/2c5f842a102c
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 25
Looping in Verdi for some documentation love.
Maybe Jennifer has something to add here. I've had conversations with Verdi who has some excellent thoughts on how to make this change more impactful for the end user. Perhaps we should have a quick meeting to discuss.
Flags: needinfo?(jboriss)
Hey(In reply to Dão Gottwald [:dao] from comment #0)
> Created attachment 779654 [details] [diff] [review]
> patch
> 
> We can just link to SUMO until bug 492557 is done. It's wanted for bugs such
> as bug 513165, bug 513168, bug 519937.

This seems to have just landed without a ui-review.  For any user-facing interface-level change, that review needs to happen.  Verdi, Grimes, and others have been researching the help menu and have a proposal based on SUMO feedback for its ordering that I see no reason not to support, but it doesn't include Keyboard Shortcuts as a top-level menu item.  Unless there's some info behind this bug I'm not aware of, I think we should back this out and defer to Grimes and Verdi on ordering.
Flags: needinfo?(jboriss) → needinfo?(dao)
(Assignee)

Comment 9

4 years ago
It's unclear to me why the current position in the Help menu would be problematic, let alone problematic enough to warrant a backout. Also, this couldn't be backed out without something replacing it, because other bugs depend on it.

I'd like to see the thoughts mentioned in comment 7 filed as new bugs, once they're settled. However, please note again that the current action of loading SUMO's keyboard shortcuts page was supposed to be replaced by bug 492557 eventually, where SUMO wouldn't really be involved. You might want to chime in there if this conflicts with your ideas.
Flags: needinfo?(dao)
This should just be part of the main 'Help' page... not a separate Menu item.

https://support.mozilla.org/en-US/products/firefox?as=u&utm_source=inproduct
(Assignee)

Comment 11

4 years ago
That doesn't sound like it would make this "more impactful" (comment 7) nor does it take bug 492557 into account.
If it's a real big deal, why not put more resources on bug 492557?
I agree, since this landed without UI review, UX / Verdi already have a clear forward plan for the help menu (which I've need infoed Verdi about)and Bug 492557 seems to be the long term solution for making keyboard shortcuts more visible, we should back this out and focus on that bug. This is over rotating on something that isn't time critical and we can take a release or two to make 492557 perfect.
Flags: needinfo?(mverdi)

Comment 14

4 years ago
Support and User Advocacy met with UX and came up with a plan for the Help menu that we'd like to implement (Bug 915908). Initially this started as a conversation around whether or not to have a help menu in Australis (and just link to SUMO) but, since we're going ahead with a help menu, it turned into rethinking what should be in the help menu. In that conversation we focused on actionable things with the highest impact. I don't think Keyboard Shortcuts fits in that case. 

Certainly there are some really helpful keyboard shortcuts, but most of them are not needed by most users. Take a look at what we have documented on SUMO https://support.mozilla.org/en-US/kb/keyboard-shortcuts-perform-firefox-tasks-quickly I don't believe the discoverability of the shortcut for selecting tab #8, for example, is of equal importance as things like resetting Firefox or learning about your browser's performance (FHR). Keyboard shortcuts can easily be found with a quick search http://mzl.la/187h8q2 (which is probably where the users most likely to use them, look for them). 

I think we should back this out as suggested in comment 8.
Flags: needinfo?(mverdi)
(Assignee)

Comment 15

4 years ago
(In reply to Verdi [:verdi] from comment #14)
> Certainly there are some really helpful keyboard shortcuts, but most of them
> are not needed by most users.

True. Keyboard shortcuts are an accessibility feature that doesn't affect most people. That doesn't mean the menu item should go away, though. Do you think it will misdirect people that don't need it?

> Keyboard shortcuts can easily be found with a quick search
> http://mzl.la/187h8q2 (which is probably where the users most likely to use
> them, look for them). 

No, they're currently exposed quite prominently right in the menus. That's changing. Please see the bugs that I mentioned multiple times now.
This landing without ui-review was not ideal.

It's not clear to me whether the best way forward is backing this (and the bugs it blocks) out, or maintaining the status quo until we have bug 492557. I don't quite understand the downsides to maintaining the status quo.

Boriss/Tyler, if you want to push for a backout here, can you propose that to firefox-dev with your reasoning?
(Assignee)

Updated

4 years ago
Blocks: 387849
You need to log in before you can comment on or make changes to this bug.