Display help tooltips for all visible UI elements when '?' is pressed (ie an inline UI guide)

RESOLVED FIXED

Status

Tree Management
Treeherder
P3
normal
RESOLVED FIXED
2 years ago
5 months ago

People

(Reporter: emorley, Assigned: jfrench)

Tracking

(Blocks: 1 bug)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(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.
(Reporter)

Comment 1

2 years ago
For a pretty cool example of this, go to https://reviewable.io 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.
(Assignee)

Comment 2

2 years ago
Pretty cool :)
(Assignee)

Comment 3

2 years ago
I'll also try out Reviewable on a low horsepower machine I have, I'm curious to see how that overlay behaves.
(Assignee)

Comment 4

2 years ago
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 :)
(Assignee)

Updated

2 years ago
See Also: → bug 1207598
(Assignee)

Comment 5

2 years ago
Created attachment 8665262 [details]
onscreenShortcutsProposed

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).
(Assignee)

Comment 6

2 years ago
Created attachment 8665503 [details] [review]
PR 1005 - onscreen shortcuts

PR for onscreen shortcuts for review. :)
Attachment #8665262 - Attachment is obsolete: true
Attachment #8665503 - Flags: review?(emorley)
(Reporter)

Comment 7

2 years ago
Comment on attachment 8665503 [details] [review]
PR 1005 - onscreen shortcuts

Left some comments :-)
Attachment #8665503 - Flags: review?(emorley)
(Assignee)

Comment 8

2 years ago
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)
(Reporter)

Comment 9

2 years ago
Comment on attachment 8665503 [details] [review]
PR 1005 - onscreen shortcuts

Thanks - please squash before landing :-)
Attachment #8665503 - Flags: review?(emorley) → review+

Comment 10

2 years ago
Commit pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/ca700dce6f9831b24bf49dbf5e223534413d3d8f
Bug 1141569 - Display onscreen keyboard shortcuts
(Assignee)

Comment 11

2 years ago
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.
(Reporter)

Updated

2 years ago
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.
(Assignee)

Comment 13

2 years ago
I had a similar error to get Travis passing, where I needed to add an entry to karma
https://github.com/mozilla/treeherder/blob/28d5708d7f285e82ca03b5c97608daa8d0c42cb2/tests/ui/config/karma.conf.js#L26

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.
https://github.com/mozilla/treeherder/blob/28d5708d7f285e82ca03b5c97608daa8d0c42cb2/ui/js/userguide.js#L3
A s/userguide.app/userguide/ fixed it - PR in bug 1209523.
(Assignee)

Comment 15

2 years ago
Verified 'part1' - Display onscreen keyboard shortcuts PR1005, working fine on stage.
Can this be closed?
Flags: needinfo?(emorley)
(Reporter)

Updated

5 months ago
Assignee: nobody → tojonmz
Status: NEW → RESOLVED
Last Resolved: 5 months ago
Flags: needinfo?(emorley)
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.