Closed Bug 1141569 Opened 9 years ago Closed 7 years ago

Display keyboard shortcuts when '?' is pressed


(Tree Management :: Treeherder, defect, P3)



(Not tracked)



(Reporter: emorley, Assigned: jfrench)




(1 file, 1 obsolete file)

(In reply to Ed Morley [:edmorley] from bug 1125264 comment #0)
> * Lack of full help/documentation:
>   - Docs can get out of date really fast, so I'd prefer something like an
> in-app overlay that explains the various features (even something as simple
> as showing the contents of all the existing button/toolbar tooltips next to
> their respective elements).

(In reply to Jonathan French (:jfrench) from bug 1137755 comment #1)
> Ed had a great idea recently for in-app help context help for all key ui
> elements(+). Possibly exposed as a toggle shortcut ("?" anyone?) which flips
> all these in-app balloon annotations on/off en masse. And we'd give that
> shortcut prominence in the Help menu, and turn the current Help icon into a
> drop down to either launch Help, or turn on in-app annotations.
> The in-app annotations would provide a slightly more in-depth explanation,
> and you'd find the item immediately since the help balloon would appear
> right beside the ui you are trying to find.
> For multi-paragraph detail on functionality, I could see additions to
> ReadTheDocs or similar, which could be linked from those in-app help
> balloons.
For a pretty cool example of this, go to and press the F1 key and then click on the highlighted buttons to see help text. Or even better, open a review on reviewable and do the same thing - explains all of the UI.
Pretty cool :)
I'll also try out Reviewable on a low horsepower machine I have, I'm curious to see how that overlay behaves.
The Reviewable overlay seems to work pretty well on a low gfx/cpu capability machine. I tried an old Win box with minimum memory and GeForce6200 with both Release and Chrome. It seems quick enough to be usable :)
See Also: → 1207598
Attached image onscreenShortcutsProposed (obsolete) —
With the recent bug/user queries about shortcuts, I started mocking up the first part, a shortcut overlay for '?' (or whatever keyboard we'd like to hook it to).
PR for onscreen shortcuts for review. :)
Attachment #8665262 - Attachment is obsolete: true
Attachment #8665503 - Flags: review?(emorley)
Comment on attachment 8665503 [details] [review]
PR 1005 - onscreen shortcuts

Left some comments :-)
Attachment #8665503 - Flags: review?(emorley)
Comment on attachment 8665503 [details] [review]
PR 1005 - onscreen shortcuts

I have a CI nit to fix, but this can get re-reviewed at this point I think :)
Attachment #8665503 - Flags: review?(emorley)
Comment on attachment 8665503 [details] [review]
PR 1005 - onscreen shortcuts

Thanks - please squash before landing :-)
Attachment #8665503 - Flags: review?(emorley) → review+
The onscreen shortcuts via '?' are merged per above. The css overlay panel support is there in treeherder-global.css for any eventual F1 inline UI help, which this bug represents. Excerpting my PR description here for reference for that work:

> unlike Github we set two properties on head,body, and once we get to the point where we are
> displaying inline Help for all UI elements, I think we'll need to ng-class override these when
> onscreenOverlayShowing = true. Otherwise the user won't be able to access the scroll bar
> to navigate down the page.
Depends on: 1209523
Unfortunately grunt build is failing with 'Parse Error' (bug ) - this is easily fixed with a closing bracket.

However minification isn't being performed on the user guide JS assets - they need to be inside a build block in userguide.html and then listed in Gruntfile.js, like we do for logviewer.html etc. I tried added those quickly now, but then get:

Error: [$injector:nomod] Module 'userguide' is not available! You either misspelled the module name or forgot to load it. If registering a module ensure that you specify the dependencies as the second argument.
I had a similar error to get Travis passing, where I needed to add an entry to karma

I wonder if we need to specify 'treeherder' as a dependency (2nd argument) here for Grunt? ie. ['treeherder']. It didn't seem needed though for Travis at the time.
A s/ fixed it - PR in bug 1209523.
Verified 'part1' - Display onscreen keyboard shortcuts PR1005, working fine on stage.
Can this be closed?
Flags: needinfo?(emorley)
Assignee: nobody → tojonmz
Closed: 7 years ago
Flags: needinfo?(emorley)
Resolution: --- → FIXED
Updating this bug for posterity, to describe the scope of what we did and didn't do.
Summary: Display help tooltips for all visible UI elements when '?' is pressed (ie an inline UI guide) → Display keyboard shortcuts when '?' is pressed
You need to log in before you can comment on or make changes to this bug.