Safari 3.1 large-caps selector problem

[GoogleCodeExporter]

Hi Dean,

this is a crazy issue. It hits us permanently in the tests for KSS (Kinetic
Stylesheets), where we use base2 for css selection.

The issue happens on Safari 3.1 (and Chrome), and only if one selects for a
class or id with at lease one *uppercase character* in it.

When I tried to reproduce it in a test page, I realized that the following
conditions are also needed for the error to happen:

- querySelectorAll has to be used. It seems querySelector works fine
  in the same case.

- The page must not have a <!DOCTYPE ... declaration in the beginning.
  I tried with more doctypes and if it's present, the problem does
  not occur.

I attached a test html, that needs to be put into the checked out trunk,
into the directory test/DOM/selectors-api.

Thanks in advance.

Original issue reported on code.google.com by reebal...@gmail.com on 12 Sep 2008 at 12:25

Attachments:

• safari-31-test.html

[GoogleCodeExporter]

It's fixable. :-)

Original comment by dean.edw...@gmail.com on 18 Sep 2008 at 1:19

• Changed state: Accepted
• Added labels: Browser-Safari, Module-base2.DOM

[GoogleCodeExporter]

This bug is serious enough to delay release of version 1.0.

Original comment by dean.edw...@gmail.com on 18 Sep 2008 at 2:18

[GoogleCodeExporter]

Bug is fixed now in Safari nightlies. (According to John Resig)

Original comment by got...@bubblenet.be on 2 Feb 2009 at 10:29

[GoogleCodeExporter]

That sounds really good, I wonder if the safari-31-test.html runs green on the
nightly version. ("Wonder" means I have no mac where I could easily test right 
now.)

Original comment by reebal...@gmail.com on 2 Feb 2009 at 10:52

[GoogleCodeExporter]

[deleted comment]

[GoogleCodeExporter]

The next selectors still are red in Safari Version 3.2.1 (4525.27.1), but all 
are lime in the latest WebKit.

TEST-s31-1 p a {background-color: lime;}
div#TEST-s31-2 p a {background-color: lime;}
div#tEst-s31-3 p a {background-color: lime;}
p.TEST-s31-class a {background-color: lime;}

Tested against version 1.0 (http://base2.googlecode.com/svn/version/1.0/)

Original comment by doek...@gmail.com on 3 Feb 2009 at 4:51

[GoogleCodeExporter]

A fix for this will be in the next release of base2.DOM.

Original comment by dean.edw...@gmail.com on 6 Feb 2009 at 5:48

• Changed state: Started