Bug 32609 – Web Inspector: Make Elements tree, properties section and resources view monospace.

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

[Pavel Feldman]

Created attachment 44970 [details]
[IMAGE] Screen 1

See screenshots attached.

[Pavel Feldman]

Created attachment 44971 [details]
[IMAGE] Screen 2

[Pavel Feldman]

Created attachment 44972 [details]
[PATCH] Proposed change.

[WebKit Review Bot]

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

[Pavel Feldman]

Created attachment 44974 [details]
[PATCH] Same with fix for windows platform and console prompt.

[WebKit Review Bot]

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

[Pavel Feldman]

Created attachment 44976 [details]
[IMAGE] Win / Chromium looks

[Timothy Hatcher]

Comment on attachment 44974 [details]
[PATCH] Same with fix for windows platform and console prompt.


> +                span.addStyleClass("console-formatted-trace source code");

Should be source-code.


> +    white-space: pre;

This should be pre-wrap, pre is pretty dangerous.


> +    font-family: Monaco, monospace;

Some people are going to ask for Menlo on Snow Leopard… But alti-alias monospace is pretty bad in my book. So I think Monaco is good.

Fix the class name before landing.

[Pavel Feldman]

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	WebCore/ChangeLog
	M	WebCore/inspector/front-end/ConsoleView.js
	M	WebCore/inspector/front-end/ElementsPanel.js
	M	WebCore/inspector/front-end/PropertiesSection.js
	M	WebCore/inspector/front-end/ResourceView.js
	M	WebCore/inspector/front-end/inspector.css
Committed r52203

[Pavel Feldman]

(landed with all comments addressed)