Closed
Bug 853181
Opened 12 years ago
Closed 12 years ago
Text overflow of scripts list is not full width of section
Categories
(DevTools :: Debugger, defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 22
People
(Reporter: basta, Assigned: vporof)
References
Details
Attachments
(1 file, 1 obsolete file)
4.17 KB,
patch
|
past
:
review+
|
Details | Diff | Splinter Review |
http://cl.ly/image/4145121j0R19
The text ends before the splitter.
Assignee | ||
Comment 2•12 years ago
|
||
Comment 3•12 years ago
|
||
Comment on attachment 728605 [details] [diff] [review]
v1
Review of attachment 728605 [details] [diff] [review]:
-----------------------------------------------------------------
My only concern is that we are papering over bug 850421 with the substitution in this patch and I haven't looked into that, yet. Could we just fix the overflow bit here and move that change to bug 850421?
Attachment #728605 -
Flags: review?(past)
Assignee | ||
Comment 4•12 years ago
|
||
(In reply to Panos Astithas [:past] from comment #3)
Ok, sure. I was thinking that bug has more complex ramifications (like maybe a platform bug?) so the "app://" substitution here was just a temporary hack. But I'm fine either way.
Assignee | ||
Comment 5•12 years ago
|
||
Removed "app://" substitution.
Attachment #728605 -
Attachment is obsolete: true
Attachment #729609 -
Flags: review?(past)
Updated•12 years ago
|
Attachment #729609 -
Flags: review?(past) → review+
Assignee | ||
Updated•12 years ago
|
Whiteboard: [land-in-fx-team]
Comment 6•12 years ago
|
||
Whiteboard: [land-in-fx-team] → [fixed-in-fx-team]
Assignee | ||
Updated•12 years ago
|
Priority: -- → P2
Comment 7•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 22
Updated•7 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•