If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Shorten onscreen time for non-sticky notifications

VERIFIED FIXED

Status

Tree Management
Treeherder
P4
minor
VERIFIED FIXED
3 years ago
2 years ago

People

(Reporter: jfrench, Assigned: jfrench)

Tracking

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
As discussed in channel just now, I'm thinking we might be able to get away with a slightly shorter persistence to the non-sticky thNotify error balloons.

eg. perhaps we can get away with 4s.

It might reduce the pain a bit for retrigger messages, before something like bug 1148664 gets fixed where we address the location of the messages so they don't tile all over the place.

Hopefully it will still give enough time to be readable.
(Assignee)

Updated

3 years ago
Assignee: nobody → tojonmz
Status: NEW → ASSIGNED
(Assignee)

Updated

3 years ago
Summary: Shorten the 5000ms thNotify.send non-sticky persistence for error messages → Shorten onscreen time for non-sticky notifications
(Assignee)

Comment 1

3 years ago
Created attachment 8588021 [details] [review]
treeherder-ui-PR#451

Please see above PR for review and status. Will flag Ed for review once he's marked back on Mon.
(Assignee)

Updated

3 years ago
Attachment #8588021 - Flags: review?(emorley)

Comment 2

3 years ago
Comment on attachment 8588021 [details] [review]
treeherder-ui-PR#451

wfm :-)
Attachment #8588021 - Flags: review?(emorley) → review+

Comment 3

3 years ago
Commit pushed to master at https://github.com/mozilla/treeherder-ui

https://github.com/mozilla/treeherder-ui/commit/2f2ec7bfb303b3ab6150e1d109d46714e1b27d6a
Bug 1150905 - Shorten onscreen time for non-sticky notifications
(Assignee)

Comment 4

3 years ago
Marking fixed per above merge. I'll verify on the next push to stage/prod.
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
(Assignee)

Comment 5

3 years ago
Verified fixed on stage.
Status: RESOLVED → VERIFIED

Comment 6

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

https://github.com/mozilla/treeherder/commit/b73790b74ce7a13865fde2b5733c09c4c3512533
Bug 1150905 - Shorten onscreen time for non-sticky notifications
You need to log in before you can comment on or make changes to this bug.