Closed
Bug 1153807
Opened 10 years ago
Closed 10 years ago
Job tooltips missing a space
Categories
(Tree Management :: Treeherder, defect, P3)
Tree Management
Treeherder
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: emorley, Assigned: emorley)
Details
Attachments
(1 file)
The tooltips shown when hovering over a job, eg:
<span class="btn job-btn btn-green key1707439 filter-shown btn-lg-xform selected-job" data-jmkey="key1707439" ignore-job-clear-on-click="" title="CPP Unit Testssuccess - 10 mins (C++ Unitary Tests)">Cpp</span>
Expected:
"CPP Unit Tests success - 10 mins (C++ Unitary Tests)"
Actual:
"CPP Unit Testssuccess - 10 mins (C++ Unitary Tests)"
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → emorley
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•10 years ago
|
||
Attachment #8591649 -
Flags: review?(tojonmz)
Comment 2•10 years ago
|
||
Comment on attachment 8591649 [details] [review]
Fix whitespace in job tooltips & reduce clutter
Thank you Ed :) The branch looks nice also, here in a local server.
Attachment #8591649 -
Flags: review?(tojonmz) → review+
Comment 3•10 years ago
|
||
Commit pushed to master at https://github.com/mozilla/treeherder-ui
https://github.com/mozilla/treeherder-ui/commit/eb60e492205605acc338a70807dddbd603203f68
Bug 1153807 - Fix whitespace in job tooltips & reduce clutter
The tooltips for completed jobs were missing a space between the job
name and the job result. In addition, the "check job details panel for
eta" string is unnecessary, particularly now people are used to not
having the eta there.
Assignee | ||
Comment 4•10 years ago
|
||
Thanks :-)
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 5•10 years ago
|
||
Commit pushed to master at https://github.com/mozilla/treeherder
https://github.com/mozilla/treeherder/commit/fe611c12ab61060272a745c8fcf3c9f1cdbf71bb
Bug 1153807 - Fix whitespace in job tooltips & reduce clutter
The tooltips for completed jobs were missing a space between the job
name and the job result. In addition, the "check job details panel for
eta" string is unnecessary, particularly now people are used to not
having the eta there.
You need to log in
before you can comment on or make changes to this bug.
Description
•