Bug 113476

Summary:

Web Inspector: [CSS] One rule appears multiple times in sidebar.

Product:

WebKit

Reporter:

Eugene Klyuchnikov <eustas>

Component:

Web Inspector (Deprecated)

Assignee:

Alexander Pavlov (apavlov) <apavlov>

Status:

RESOLVED INVALID

Severity:

Normal

CC:

apavlov, eustas, keishi, loislo, pfeldman, pmuellr, vsevik, web-inspector-bugs, yurys

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
Demo HTML	none
----	none
Demo screenshot	none

Eugene Klyuchnikov

Reported 2013-03-27 22:10:24 PDT

Created attachment 195481 [details] Demo HTML In some cases one rule appears several times in sidebar. Moreover it overrides own values. See demo HTML and screenshot in attachemts.

Attachments
Demo HTML (485 bytes, text/html) 2013-03-27 22:10 PDT, Eugene Klyuchnikov	no flags	Details
---- (22.80 KB, image/png) 2013-03-27 22:10 PDT, Eugene Klyuchnikov	no flags	Details
Demo screenshot (126.27 KB, image/png) 2013-03-27 22:19 PDT, Eugene Klyuchnikov	no flags	Details
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Eugene Klyuchnikov

Comment 1 2013-03-27 22:10:59 PDT

Created attachment 195482 [details] ----

Eugene Klyuchnikov

Comment 2 2013-03-27 22:19:43 PDT

Created attachment 195483 [details] Demo screenshot

Alexander Pavlov (apavlov)

Comment 3 2013-03-28 01:06:31 PDT

I'm not sure if we want to fix this at all (but this opinion is by no means authoritative), since this is what WebCore reports to us, and this is how exactly it performs rule matching. Since .a, .b, .c { X } is equivalent to .a { X } .b { X } .c { X } and all three selector groups (".a", ".b", ".c") match the selected element independently (notice that all of them are black, not grayed-out), the rule is reported thrice. Moreover, selector groups might have different specificities, so the same rule may get reported in different positions inside the cascade, and this may turn useful sometimes. I believe we might want to eliminate repetitive consecutive matches for the same rule, though.

Eugene Klyuchnikov

Comment 4 2013-03-28 01:38:36 PDT

(In reply to comment #3) > I'm not sure if we want to fix this at all (but this opinion is by no means authoritative), since this is what WebCore reports to us, and this is how exactly it performs rule matching. Oh, I see. Thank you for explanation.

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