dojo.query picky vis a vis whitespace

[dojotoolkit@…]

Given:

<ul>
  <li class="myClass"><a href="#">Link</a></li>
</ul>

This works:

dojo.query('li.myClass a');

This fails (note the extra space):

dojo.query('li.myClass  a');

[dylan]

to keep things lean and as fast as possible with this, I sort of think this is a good idea? Kicking to Alex for his thoughts on performance impact of normalizing whitespace on dojo.query queries.

[dylan]

note that the following fails:

dojo.query('div,p');

whereas this works:

dojo.query('div, p');

People writing css are used to having their whitespace auto-normalized.

[dylan]

(In [8724]) refs #2423, add test showing dojo.query whitespace pickiness

[dylan]

Added a test.. here's the failure:

     _AssertFailure: http://dojo.localdomain/dojo/_base/_loader/bootstrap.js:281 doh._AssertFailure: assertEqual() failed: expected |2| but got |0|
 doh._AssertFailure
     ERROR IN:
  (function anonymous(t) {t.is(2, dojo.query(".foo,.bar").length);})

[dylan]

more info in #3197

[alex]

(In [9059]) make query() more resiliant in whitespace parsing for compund queries. Refs #2423

[tk]

Test appears to pass here: ​http://archive.dojotoolkit.org/nightly/dojotoolkit/dojo/tests/_base/query.html however, firebug isnt included in the page so not for sure how a failure would look...

Should this be closed now? ( I only have IE to work with here... so cant just check firebug in FF)

-Karl

[Douglas Hays]

It seems this is fixed, from firebug I see:

PASSED test: t.is(2, (dojo.query('.foo, .bar')).length);

PASSED test: t.is(2, (dojo.query('.foo,.bar')).length);

[alex]

there are still some nagging issues w/ whitespace in query() which aren't solved yet. I'm re-opening but targeting for 1.0.

[alex]

(In [11174]) complete replacement for the tokenization strategy of the query engine. Moved from ad-hoc substring searching to an AST generation/consumption system. The query engine should be significantly more robust in the face of whitespace and funky formatting than before. Code size change should be minimal despite the large-scale change. Expanded tests and a first stab at inline docs for dojo.query. All unit tests pass on FF2, Safari3, and IE 6 and 7. Fixes #4365. Fixes #4640. Fixes #2423.

[alex]

(In [11175]) re-enable query optimizer. Refs #2423