855518 - Make monospace font readable and consistent across devtools

Status: RESOLVED DUPLICATE of bug 905312

(DevTools :: Inspector, defect)

Description

[Victor Porof [:vporof][:vp]]

After bug 836233, the markup view text seems a bit smaller than usual (vertically undersized), and is especially hard to read (on OS X at least). Compared to the source editor text, it's about half the size, from what I can tell.

Comment 1

[Heather Arthur [:harth]]

Attachment: WIP, increase "devtools-monospace" size on OS X, use in net panel

Trying some stuff out. It seems to be a decent size on Windows and Linux. The Network panel had it's own monospace font, so I gave it the devtools-monospace class instead.

Comment 2

[Paul Rouget [:paul]]

(I'd like to review that when it's ready)

Comment 3

[Heather Arthur [:harth]]

Comment on attachment 791583 [details] [diff] [review]
WIP, increase "devtools-monospace" size on OS X, use in net panel

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

This does a few things:

* Netmonitor now uses "devtools-monospace" class instead of hard-coding its monospace font
* Font panel now uses "devtools-monospace" and doesn't define its own sizes.
* On OS X, monospace font size is increased to %110. The other platforms look alright size-wise (aside from the font panel).
* Fix margin issue in font panel.

Comment 4

[Heather Arthur [:harth]]

Attachment: font-size.patch

Patch, goes along with notes in last comment ^.


Try builds: http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/[e-mail]-7635c7cbcde8/

Comment 5

[Heather Arthur [:harth]]

Comment on attachment 792363 [details] [diff] [review]
font-size.patch

Gonna tackle web console too. I'll upload a new patch.