Closed
Bug 1161132
Opened 9 years ago
Closed 9 years ago
Allow user resizing for console.dir output
Categories
(DevTools :: Console, defect, P2)
Tracking
(firefox40 fixed)
RESOLVED
FIXED
Firefox 40
Tracking | Status | |
---|---|---|
firefox40 | --- | fixed |
People
(Reporter: jfong, Assigned: jfong)
References
Details
(Whiteboard: [polish-backlog])
Attachments
(1 file)
884 bytes,
patch
|
bgrins
:
review+
|
Details | Diff | Splinter Review |
We need a way to vertically resize the console.dir output for larger screens.
Assignee | ||
Comment 1•9 years ago
|
||
Should we add devedition-40 whiteboard info to this bug?
Flags: needinfo?(jgriffiths)
Comment 2•9 years ago
|
||
For some background, we removed the attempt to resize this based on the current window height (which didn't always work) in Bug 1157944 (see Comment 8 - Comment 12). We discussed being able to allow user resizing of this output (sort of like how you can resize textareas).
Depends on: 1157944
Updated•9 years ago
|
Summary: Fix CSS for console.dir output → Allow user resizing for console.dir output
Comment 3•9 years ago
|
||
(In reply to Jen Fong-Adwent [:ednapiranha] from comment #1) > Should we add devedition-40 whiteboard info to this bug? Still got a week to go, why not!
Flags: needinfo?(jgriffiths)
Whiteboard: [devedition-40]
Updated•9 years ago
|
Priority: -- → P2
Assignee | ||
Comment 4•9 years ago
|
||
Setting the min-height to 200px instead of 100px fixes the scroll situation where the top or bottom of the iframe cuts off during vertical resize. Let me know if you think anything else needs to be adjusted or we're good to go.
Attachment #8602867 -
Flags: review?(bgrinstead)
Comment 5•9 years ago
|
||
Comment on attachment 8602867 [details] [diff] [review] Bug1161132.patch Review of attachment 8602867 [details] [diff] [review]: ----------------------------------------------------------------- LGTM
Attachment #8602867 -
Flags: review?(bgrinstead) → review+
Assignee | ||
Comment 6•9 years ago
|
||
Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=4ed35dffdb1e
Updated•9 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Comment 8•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/90bfa486922a
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 40
Updated•9 years ago
|
Whiteboard: [devedition-40] → [polish-backlog]
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•