1077523 - Enable CSS source maps by default

Status: RESOLVED FIXED

(DevTools :: Style Editor, defect)

Description

[Heather Arthur [:harth]]

CSS source map support has been in for many releases and many bugs fixed. I think it's time to enable it by default as most people want to see the source they authored and not the generated one.

Comment 1

[Heather Arthur [:harth]]

Attachment: CSS source maps on by default

Comment 2

[Heather Arthur [:harth]]

^ That patch also makes the inspector context menu item "Show original sources" a checkbox, since seeing "Show CSS sources" would be confusing (I wouldn't know what that meant).

Comment 3

[Brian Grinstead [:bgrins]]

Comment on attachment 8502614 [details] [diff] [review]
CSS source maps on by default

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

Sounds good to me! Pushed to try: https://tbpl.mozilla.org/?tree=Try&rev=c450255fa707

Comment 4

[Heather Arthur [:harth]]

Attachment: Enable sourcemaps + tooltip fix

Thanks for pushing to try! There was a failure. Turns out sometimes we were displaying the line number in the tooltip for a location link. Small fix so that we always display it.

Try:
https://tbpl.mozilla.org/?tree=Try&rev=3b112b9866d3

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/integration/fx-team/rev/0218c2f545e8

Comment 6

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/0218c2f545e8