Closed
Bug 1109333
Opened 10 years ago
Closed 10 years ago
Selected blackboxed sources in the debugger aren't correctly styled
Categories
(DevTools :: Debugger, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 37
People
(Reporter: vporof, Assigned: vporof)
Details
Attachments
(2 files, 1 obsolete file)
43.33 KB,
image/png
|
Details | |
1.26 KB,
patch
|
bgrins
:
review+
|
Details | Diff | Splinter Review |
Two issues: * labels are hard to read * breakpoints should be hidden
Assignee | ||
Comment 1•10 years ago
|
||
Should backport to aurora too.
Comment 2•10 years ago
|
||
(In reply to Victor Porof [:vporof][:vp] from comment #1) > Created attachment 8534001 [details] [diff] [review] > v1 > > Should backport to aurora too. This goes at least as far back as 34, but yes, we may as well uplift a fix to aurora
Comment 3•10 years ago
|
||
Comment on attachment 8534001 [details] [diff] [review] v1 Review of attachment 8534001 [details] [diff] [review]: ----------------------------------------------------------------- ::: browser/themes/shared/devtools/debugger.inc.css @@ +99,5 @@ > -moz-image-region: rect(0px,64px,32px,32px); > } > } > > +#sources .dbg-source-item.black-boxed { I think it would work to change just the next other two selectors instead of prepending .dbg-source-item onto each (so this one could be left alone). Am I missing something here? @@ +104,4 @@ > color: rgba(128,128,128,0.4); > } > > +#sources .selected .dbg-source-item.black-boxed { #sources .selected .black-boxed { @@ +108,4 @@ > color: rgba(255,255,255,0.4); > } > > +#sources .dbg-source-item.black-boxed ~ .dbg-breakpoint { #sources .black-boxed ~ .dbg-breakpoint {
Attachment #8534001 -
Flags: review?(bgrinstead)
Assignee | ||
Comment 4•10 years ago
|
||
(In reply to Brian Grinstead [:bgrins] from comment #3) > Comment on attachment 8534001 [details] [diff] [review] > v1 > Sure, I just wanted to be more specific.
Assignee | ||
Comment 5•10 years ago
|
||
Attachment #8534001 -
Attachment is obsolete: true
Attachment #8534014 -
Flags: review?(bgrinstead)
Updated•10 years ago
|
Attachment #8534014 -
Flags: review?(bgrinstead) → review+
Assignee | ||
Comment 6•10 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/6588279177dd
Whiteboard: [fixed-in-fx-team]
Comment 7•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/6588279177dd
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 37
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•