Closed Bug 853181 Opened 11 years ago Closed 11 years ago

Text overflow of scripts list is not full width of section

Categories

(DevTools :: Debugger, defect, P2)

22 Branch
x86
macOS
defect

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 22

People

(Reporter: basta, Assigned: vporof)

References

Details

Attachments

(1 file, 1 obsolete file)

http://cl.ly/image/4145121j0R19

The text ends before the splitter.
On the latest nightly
Version: unspecified → 22 Branch
Attached patch v1 (obsolete) — Splinter Review
Assignee: nobody → vporof
Status: NEW → ASSIGNED
Attachment #728605 - Flags: review?(past)
Blocks: 854174
Depends on: 762160
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)
(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.
Attached patch v1.1Splinter Review
Removed "app://" substitution.
Attachment #728605 - Attachment is obsolete: true
Attachment #729609 - Flags: review?(past)
Attachment #729609 - Flags: review?(past) → review+
Whiteboard: [land-in-fx-team]
Priority: -- → P2
https://hg.mozilla.org/mozilla-central/rev/94c03a12288b
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 22
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: