761231 - Font for the Debugger panels should be message-box

Status: RESOLVED FIXED

(DevTools :: Debugger, defect)

Description

[Jared Wein [:jaws] (please needinfo? me)]

I've noticed that the font for the various panels within the Debugger is list-box, which looks pretty ugly on Windows and feels Windows 3.1-ish.

If we switch to font:message-box we'll get Segoe UI on Windows and the respective UI fonts for the other platforms as well.

Comment 1

[Panos Astithas (he/him) [:past] (please ni?)]

Attachment: Patch

This should do it.

Comment 2

[Dão Gottwald [:dao]]

Shouldn't this be message-box by default? What happens if you just remove font: -moz-list;?

Comment 3

[Panos Astithas (he/him) [:past] (please ni?)]

Attachment: Patch v2

(In reply to Dão Gottwald [:dao] from comment #2)
> Shouldn't this be message-box by default? What happens if you just remove
> font: -moz-list;?

Good point, it looks identical. We probably used to require them when the UI was in HTML.

Comment 4

[Panos Astithas (he/him) [:past] (please ni?)]

https://hg.mozilla.org/integration/fx-team/rev/233a5180da19

Comment 5

[Tim Taubert [:ttaubert] (inactive)]

https://hg.mozilla.org/mozilla-central/rev/233a5180da19

Comment 6

[Rob Campbell [:rc] (:robcee)]

Comment on attachment 630156 [details] [diff] [review]
Patch v2

[Approval Request Comment]
Bug caused by (feature/regressing bug #): New feature.
User impact if declined: Fonts may look funny.
Testing completed (on m-c, etc.): on m-c
Risk to taking this patch (and alternatives if risky): Style only.
String or UUID changes made by this patch:

Comment 7

[Panos Astithas (he/him) [:past] (please ni?)]

https://hg.mozilla.org/releases/mozilla-aurora/rev/d2222d108e06

Comment 8

[Ioana (away)]

Verified as fixed on Firefox 15.0 beta 3 (20120731150526).