Closed Bug 1033266 Opened 10 years ago Closed 10 years ago

Modifier for multiple job selection using mouse is different from TBPL

Categories

(Tree Management :: Treeherder, defect, P2)

Product:
Tree Management
Component:
Treeherder
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: emorley, Assigned: jfrench)

References

(
URL
)

Details

The Treeherder modifier for selecting multiple jobs (adding multiple jobs to the pin board) is "shift" rather than TBPL's "ctrl". 

Also, most OSes/other apps use ctrl to mean "add single item to current list" and shift to mean "add range to list" - so I think ctrl makes more sense here, even if we weren't worried about preserving TBPL functionality.
Summary: Keyboard modifier for multiple job selection is different from TBPL → Modifier for multiple job selection using mouse is different from TBPL
I'll have a look at this, and see what I can do.
Assignee: nobody → tojonmz
URL: https://treeherder.mozilla.org
Status: NEW → ASSIGNED
Everything seems to be behaving as expected. I've opened a PR for review
https://github.com/mozilla/treeherder-ui/pull/101
The PR has been landed on master and I've tested it on dev and everything appears to be correct. I will retest after it is pushed to production and will mark it Fixed.
It turns out we need to provide Mac support also - with Cmd+click since that was supported in TBPL on OSX. Ctrl+click on Mac just fires a context menu and the log which is not what we want. Since I'm windows only here, Cameron has addressed that with the commit below.

https://github.com/mozilla/treeherder-ui/commit/7e557a3ebaa2c0ae41498b0485b9c87645f33c4b

It's been pushed to dev/stage, and will be pushed to prod next week.
Camd has pushed to prod, and it appears to be working fine. Marking resolved fixed.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.