Closed
Bug 1150905
Opened 9 years ago
Closed 9 years ago
Shorten onscreen time for non-sticky notifications
Categories
(Tree Management :: Treeherder, defect, P4)
Tree Management
Treeherder
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: jfrench, Assigned: jfrench)
Details
Attachments
(1 file)
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•9 years ago
|
Assignee: nobody → tojonmz
Status: NEW → ASSIGNED
Assignee | ||
Updated•9 years ago
|
Summary: Shorten the 5000ms thNotify.send non-sticky persistence for error messages → Shorten onscreen time for non-sticky notifications
Assignee | ||
Comment 1•9 years ago
|
||
Please see above PR for review and status. Will flag Ed for review once he's marked back on Mon.
Assignee | ||
Updated•9 years ago
|
Attachment #8588021 -
Flags: review?(emorley)
Comment 2•9 years ago
|
||
Comment on attachment 8588021 [details] [review] treeherder-ui-PR#451 wfm :-)
Attachment #8588021 -
Flags: review?(emorley) → review+
Comment 3•9 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•9 years ago
|
||
Marking fixed per above merge. I'll verify on the next push to stage/prod.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Comment 6•9 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.
Description
•