1089020 - Hide the "cancel all jobs" button for non-try trees

Status: RESOLVED FIXED

(Tree Management :: Treeherder, defect, P2)

Description

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

Attachment: Link to pull request 249

I've clicked cancel all way too many times on m-c when I'm just trying to click the button to the left of it to access the links to mc-merge or self-serve.

Bug 1074858 wants to make the cancel all button prompt before it kills everything. 
Bug 1075473 wants to let users with is_staff go ahead and use cancel all for non-try, but just make it more obvious that it's affecting non-try stuff.

Both of those are good ideas, but there's been no activity in either bug.

I propose we just hide the buttons on non-try for now until we can get the UX a little better.

Comment 1

[Ed Morley [:emorley]]

Comment on attachment 8511430 [details] [review]
Link to pull request 249

Could you switch this to a check for the repo group "try", rather than name?
Also, does the layout look ok with the button removed?

Thank you for taking a look at this :-)

Comment 2

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

Attachment: non-try.png

Comment 3

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

Attachment: try.png

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

Comment on attachment 8511430 [details] [review]
Link to pull request 249

Edited the pull request so it checks for repository_group.name == "try".

Attached is what I see for non-try and try repos. The appearance looks fine to me.

Comment 5

[Ed Morley [:emorley]]

Comment on attachment 8511430 [details] [review]
Link to pull request 249

Looks good, thank you :-)

Comment 6

[Ed Morley [:emorley]]

This had bug 1075473 in the commit message, so the bot commented over there.

(In reply to Treeherder Bugbot from comment #2)
> Commits pushed to master at https://github.com/mozilla/treeherder-ui
> 
> https://github.com/mozilla/treeherder-ui/commit/94855c938516281d26c60ae84821375a388946f2
> Bug 1075473 - Hide the cancel all button for all repos that aren't in the
> try group.
> 
> https://github.com/mozilla/treeherder-ui/commit/e12443fad9c907e7938548516b440f6d86c72518
> Merge pull request #249 from KWierso/KILLCANCEL
> 
> Bug 1075473 - Hide the cancel all button for repos not in the 'try' group