Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#14874 closed defect (wontfix)

Camel Cased classname queries fail on lite engine

Reported by: bill Owned by: Kris Zyp
Priority: undecided Milestone: tbd
Component: Query Version: 1.7.2
Keywords: Cc:
Blocked By: Blocking:

Description

See related ticket #8775. Apparently CSS Spec denotes class selectors are supposed to be case insensitive. In tests/query/query.html:

query('.fooBar')

should match both:

<h3 class="baz foobar" title="thud">h3</h3>
<span class="fooBar baz foo"></span>

but it only matches one of them.

Tested on FF11/mac.

Uncomment the test in tests/query/query.html when this is fixed.

Change History (7)

comment:1 Changed 7 years ago by bill

In [27986]:

Make better tests for new dojo/query module, refs #12451, #14874, #14875, #14876, #14877, #14879, #14880.

comment:2 Changed 7 years ago by Kris Zyp

Resolution: wontfix
Status: newclosed

Case sensitivity is dependent on the doctype. Some doctypes denote case sensitivity of class names and some don't. I believe that for the doctype of the test page, the class names are supposed to be case sensitive.

comment:3 Changed 7 years ago by bill

OK, I read over #8775 and it does say that standards mode is supposed to be case sensitive. OTOH, it still seems to be case insensitive for tag names as shown by the other tests in query.html:

"doh.is(1, (query('span.baz')).length);",
"doh.is(1, (query('sPaN.baz')).length);",
"doh.is(1, (query('SPAN.baz')).length);",

Anyway, I'll update the test file to indicate that only acme is case-insensitive on class names in standards-mode documents.

comment:4 Changed 7 years ago by bill

In [28045]:

In standards-mode documents, except for acme, class names are supposed to be case-sensitive. Refs #8775, #14874

comment:33 Changed 7 years ago by bill

In [29415]:

use relative paths in require.toUrl(), rather than depending on weird mapping from "tests" to "dojo/tests", fixes query and dojo/data tests in built version of dojo, refs #14874 !strict

comment:34 Changed 7 years ago by bill

In [30111]:

Disable test for attribute case sensitivity in quirks mode, since it's browser dependent and we don't want to bother "fixing" it, whatever that would mean. Avoids test failure on FF. Refs #8775, #14874 on trunk.

comment:34 Changed 7 years ago by bill

In [30112]:

Disable test for attribute case sensitivity in quirks mode, since it's browser dependent and we don't want to bother "fixing" it, whatever that would mean. Avoids test failure on FF. Refs #8775, #14874 on 1.8 branch.

Note: See TracTickets for help on using tickets.