Summary: | CSS3 nth-child(n) selector fails | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ian Ellison-Taylor <ianet> | ||||||||
Component: | CSS | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | commit-queue, eric, lea, yael | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | PC | ||||||||||
OS: | Windows 7 | ||||||||||
URL: | http://tools.css3.info/selectors-test/test.html | ||||||||||
Attachments: |
|
Created attachment 86658 [details]
Simpler test case
Added a simpler test from Tab, text in the test should be green
FWIW, here's another testcase: http://jsfiddle.net/leaverou/Cum32/ Created attachment 91811 [details]
Patch.
Comment on attachment 91811 [details]
Patch.
OK.
The spec link in the ChangeLog or source would have been nice. :) Also generally our test cases use 4-space indent (like the rest of our code), but it's not a big deal. Comment on attachment 91811 [details] Patch. Clearing flags on attachment: 91811 Committed r85416: <http://trac.webkit.org/changeset/85416> All reviewed patches have been landed. Closing bug. |
Created attachment 86654 [details] nth-child(n) test case The nth-child(n) CSS3 test fails although nth-child(1n) passes. See the attached test, the background for the 4 elements should be red.