Bug 32646 – Web Inspector: Switch to 11px as default for source-code in WebKit

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector (Deprecated)

[Pavel Feldman]

Some people (including myself and Joe) think that our font is too small and things are not clear. Some of these concerns are addressed via migrating to monospace. But not all. What do you say about trying out 11px as the default size?

- Web developers typically have big enough screens
- It looks good on my MBP
- Firebug does not hesitate using large fonts and people seem to like it.
- I made it 12px monospace in Chromium (that is ~11px Monaco)

Throwing some images in shortly.

[Pavel Feldman]

Created attachment 45039 [details]
[IMAGE] 10px

[Pavel Feldman]

Created attachment 45040 [details]
[IMAGE] 11px

[Pavel Feldman]

I encourage you to do side-by-side eval of the images. For me 11px looks much more clear and fits almost as much as 10px (15 lines vs 16 lines when attached).

[Pavel Feldman]

Created attachment 45041 [details]
[PATCH] Just in case you really, really like it!

[WebKit Review Bot]

style-queue ran check-webkit-style on attachment 45041 [details] without any errors.

[Mark Rowe (bdash)]

10px Monaco should be the default for Leopard and earlier.  11px Menlo should be the default for SnowLeopard.  This matches the default size and family of monospace fonts used elsewhere in the operating system.  We should match these defaults until we provide some means for users to adjust these to their personal preference (user stylesheet perhaps?).

[Timothy Hatcher]

I agree with Mark. Monaco >10px is just too ugly to use…

This really needs to be a setting as it is highly a personal opinion.

[Timothy Hatcher]

I personally can't get use to Menlo either (being antialiased and monospaced bothers me). But it might look decent in all the places that are no monospace that use to be an antialiased font… hmm.

[Timothy Hatcher]

I'm fine with 11px on Windows. But on Mac we need to match the system.

[Timothy Hatcher]

Comment on attachment 45041 [details]
[PATCH] Just in case you really, really like it!

Where did the Monaco reference go? If this is now just using the default monospace font (Monaco/Menlo), then the default font size will be implied from WebPreferences if you just do this:

font: monospace

[Pavel Feldman]

(In reply to comment #10)
> (From update of attachment 45041 [details])
> Where did the Monaco reference go? If this is now just using the default
> monospace font (Monaco/Menlo), then the default font size will be implied from
> WebPreferences if you just do this:
> 
> font: monospace

It is .source-code in inspector.js

[Timothy Hatcher]

Lets land this and see what people think. I suspect we will need to add settings for this soon. And default to wha tthe platform standard is (Monaco, Menlo, Consolas, etc).

[Pavel Feldman]

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	WebCore/ChangeLog
	M	WebCore/WebCore.xcodeproj/project.pbxproj
	M	WebCore/inspector/front-end/inspector.css
Committed r52260