about:memory should have a larger max width

RESOLVED FIXED in mozilla16

Status

()

Toolkit
about:memory
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Gavin, Assigned: Gavin)

Tracking

Trunk
mozilla16
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

This can be easily achieved by setting class "aboutPageWideContainer" on the body. More room to see long URLs and such on wide screen monitors.
Created attachment 640001 [details] [diff] [review]
patch
Assignee: nobody → gavin.sharp
Status: NEW → ASSIGNED
Attachment #640001 - Flags: review?(n.nethercote)
Before patch: http://cl.ly/391e2B1u2H3E381F1o2U
After patch: http://cl.ly/463L1i3V3c3M0O3l3l1Z
Comment on attachment 640001 [details] [diff] [review]
patch

Review of attachment 640001 [details] [diff] [review]:
-----------------------------------------------------------------

jlebar is the CSS expert when it comes to about:memory, but I reckon I can r+ this :)
Attachment #640001 - Flags: review?(n.nethercote) → review+
I think we should make the compartment URLs appear again in the tooltips.  I miss those.

But that's a separate bug; this looks good.  :)
https://hg.mozilla.org/mozilla-central/rev/1c58f5b42d42
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla16

Comment 6

5 years ago
(In reply to Justin Lebar [:jlebar] from comment #4)
> I think we should make the compartment URLs appear again in the tooltips.  I
> miss those.
> 
> But that's a separate bug; this looks good.  :)

+1 for the tooltips, esp as there is now even more information and thus longer lines.

IMHO tooltips are only needed for lines with more than 50-60 chars (old width was 50em)

Thanks for the patch. Until now I've had the line:
body.non-verbose { max-width: 90% !important; }
in my chrome/userContent.css file.
You need to log in before you can comment on or make changes to this bug.