RESOLVED FIXED 154216
Web Inspector: CodeMirror styles needs updated for CSS variables 
https://bugs.webkit.org/show_bug.cgi?id=154216
Summary Web Inspector: CodeMirror styles needs updated for CSS variables
Timothy Hatcher
Reported 2016-02-13 10:21:16 PST
We need to style the .cm-m-css.cm-varible-2 like a property.
Attachments
Patch (2.40 KB, patch)
2016-02-13 10:34 PST, Timothy Hatcher 		no flags
DetailsFormatted DiffDiff
With Patch Applied (534.37 KB, image/png)
2016-02-13 10:34 PST, Timothy Hatcher 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2016-02-13 10:21:32 PST
<rdar://problem/24644146>
Timothy Hatcher
Comment 2 2016-02-13 10:34:02 PST
Created attachment 271287 [details] Patch
Timothy Hatcher
Comment 3 2016-02-13 10:34:35 PST
Created attachment 271288 [details] With Patch Applied
Joseph Pecoraro
Comment 4 2016-02-15 12:23:46 PST
Comment on attachment 271287 [details] Patch r=me Should we give them a different color, so they stand out? Just making them look like regular properties seems like a missed opportunity.
Timothy Hatcher
Comment 5 2016-02-15 12:42:49 PST
(In reply to comment #4) > Comment on attachment 271287 [details] > Patch > > r=me > > Should we give them a different color, so they stand out? Just making them > look like regular properties seems like a missed opportunity. Maybe we could, but I am not sure what a good color would be.
WebKit Commit Bot
Comment 6 2016-02-15 13:13:23 PST
Comment on attachment 271287 [details] Patch Clearing flags on attachment: 271287 Committed r196592: <http://trac.webkit.org/changeset/196592>
WebKit Commit Bot
Comment 7 2016-02-15 13:13:26 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.