Closed
Bug 1421107
Opened 7 years ago
Closed 7 years ago
Stop the Performance tab from wrapping at narrower widths
Categories
(Tree Management :: Treeherder: Frontend, enhancement, P3)
Tree Management
Treeherder: Frontend
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jfrench, Assigned: jfrench)
Details
Attachments
(2 files)
The Performance tab for talos jobs appears to be wrapping at modest browser widths, when it shouldn't. I'll attach a screen grab. If I recall way back when, I think we had some dynamic percentage control which calculated the max width of the parent container which prevented this. Perhaps it regressed with the bs4 upgrade.
Assignee | ||
Comment 1•7 years ago
|
||
Per screen grab.
Assignee | ||
Comment 2•7 years ago
|
||
(In reply to Jonathan French (:jfrench) from comment #0) > If I recall way back when, I think we had some dynamic percentage control > which calculated the max width of the parent container which prevented this. > Perhaps it regressed with the bs4 upgrade. This was that prior functionality https://github.com/mozilla/treeherder/pull/1002
Assignee | ||
Comment 3•7 years ago
|
||
And looks like the dynamic width control got removed in this change (ui/plugins/pluginpanel.html L-227) https://github.com/mozilla/treeherder/commit/f6258bf4d55a7763b17c3efb58f627e7d8751589
Comment 4•7 years ago
|
||
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → tojonmz
Status: NEW → ASSIGNED
Assignee | ||
Comment 5•7 years ago
|
||
Comment on attachment 8933918 [details] [review] Link to GitHub pull-request: https://github.com/mozilla/treeherder/pull/3012 For review at your leisure :)
Attachment #8933918 -
Flags: review?(cdawson)
Comment 6•7 years ago
|
||
Commit pushed to master at https://github.com/mozilla/treeherder https://github.com/mozilla/treeherder/commit/2c3b9b7370581486ed47d3a37fd9de9e624468ea Bug 1421107 - Stop perf tab wrapping at narrower widths
Updated•7 years ago
|
Attachment #8933918 -
Flags: review?(cdawson) → review+
Assignee | ||
Updated•7 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•