Closed Bug 1267998 Opened 3 years ago Closed 3 years ago

Firebug theme: better CSS for memory heap-tree

Categories

(DevTools :: Memory, defect)

defect
Not set

Tracking

(firefox49 fixed)

RESOLVED FIXED
Firefox 49
Tracking Status
firefox49 --- fixed

People

(Reporter: Honza, Assigned: Honza)

References

Details

Attachments

(1 file)

This is a follow up for bug 1266310

Just a little enhancement of the existing CSS.

Honza
Assignee: nobody → odvarko
Attached patch bug1267998.patchSplinter Review
(In reply to Tim Nguyen :ntim from comment #7)
> Comment on attachment 8745341 [details] [diff] [review]
> bug1266310.patch
> 
> Review of attachment 8745341 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Seems like the other patch landed in fx-team [0], so you'll probably need to
> make this a follow up patch.
> 
> [0]: https://hg.mozilla.org/integration/fx-team/rev/146e4f2558c9
> 
> Anyway, this looks good to me, thanks for addressing my comments!
> 
> ::: devtools/client/themes/memory.css
> @@ +524,5 @@
> >  
> >  .heap-tree-item-individuals > button {
> >    height: 10px;
> >    width: 32px;
> > +  margin: 0 auto !important;
> 
> Can you add a comment on why !important is needed?

Done in the attached patch.
Honza
Attachment #8746018 - Flags: review?(ntim.bugs)
Attachment #8746018 - Flags: review?(ntim.bugs) → review+
https://hg.mozilla.org/mozilla-central/rev/a308ab434d13
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
Depends on: 1285455
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.