Bug 74568

Summary:

Web Inspector: only the first @rule is highlighted in CSS

Product:

WebKit

Reporter:

Timothy Hatcher <timothy>

Component:

Web Inspector (Deprecated)

Assignee:

Pavel Feldman <pfeldman>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

apavlov, bweinstein, dglazkov, joepeck, keishi, loislo, pfeldman, pmuellr, rik, timothy, webkit.review.bot, yurys

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
Example from Daring Fireball	none
Patch	timothy: review+, webkit.review.bot: commit-queue-

Timothy Hatcher

Reported 2011-12-14 17:25:37 PST

Created attachment 119347 [details] Example from Daring Fireball Looking at http://daringfireball.net/css/fireball_screen.css which has a @charset rule at the top, the @import rules later in the file are not highlighted. Maybe our @rule code is expecting {}?

Attachments
Example from Daring Fireball (20.18 KB, text/css) 2011-12-14 17:25 PST, Timothy Hatcher	no flags	Details
Patch (3.26 KB, patch) 2011-12-15 13:29 PST, Pavel Feldman	timothy: review+ webkit.review.bot: commit-queue-	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Pavel Feldman

Comment 1 2011-12-15 13:29:21 PST

WebKit Review Bot

Comment 2 2011-12-15 14:48:16 PST

Comment on attachment 119491 [details] Patch Attachment 119491 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/10904409 New failing tests: inspector/syntax-highlight-css.html

Pavel Feldman

Comment 3 2011-12-19 05:35:41 PST

Note You need to log in before you can comment on or make changes to this bug.