dojo.query :selected does not work

[guest]

The following code returns EVERY input element within a form not just the input elements that are checked (which would make them type=checkbox too)

Accourding to the Dojo documentation HERE it is suppose to return only the Checked elements.

var form = dojo.byId( 'formID' );

var results = dojo.query( "input:checked", form );

I also tried this:

var form = dojo.byId( 'formID' );

var results = dojo.query( "input[type=checkbox][checked]", form );

But it did not return any elements.

[guest]

The title is a little confusing -- I meant to mention that both the :selected and the :checked are broken in this regard.

[guest]

I found the following comment in dojo.query -- I believe we should make this information known in the Documentation on Query so others like me don't waste their time trying to figure out why they don't work.

		//		Unsupported Selectors:
		//		--------------------
		//
		//		While dojo.query handles many CSS3 selectors, some fall outside of
		//		what's resaonable for a programmatic node querying engine to
		//		handle. Currently unsupported selectors include:
		//		
		//			* namespace-differentiated selectors of any form
		//			* "~", the immediately preceeded-by sibling selector
		//			* "+", the preceeded-by sibling selector
		//			* all "::" pseduo-element selectors
		//			* certain pseduo-selectors which don't get a lot of day-to-day use:
		//				* :root, :lang(), :target, :focus
		//			* all visual and state selectors:
		//				* :root, :active, :hover, :visisted, :link, :enabled, :disabled, :checked
		//			* :*-of-type pseudo selectors

[bill]

Move all milestone 1.1 tickets to 1.2, except for reopened tickets and tickets opened after 1.1RC1 was released.

[alex]

(In [14408]) adding support for the ":checked" pseudo selector. Fixes #5179. !strict

[alex]

note that the CSS 3 working draft has removed the ":selected" pseudo-class in favor of ":checked", which we now implement

[Sam Foster]

[CLA] [PATCH] Adds failing option:checked test to tests/_base/query.html

[Sam Foster]

I've attached a patch for the query test file that adds a test for :checked pseudo as applied to a select option. This passes for me in FF3.5, but fails in IE7 and Safari 4.0

I tried amending the function in query.js, like so:

"checked": function(name, condition){
	return function(elem){
		return !!d.attr(elem, d.hasAttr(elem, "checked") ? "checked" : "selected");
	}
},

but while this fixed IE, it seems to still fail in Safari? Not sure if safari is using this branch, or what's going on.

[Sam Foster]

(In [20574]) Added test for :checked psuedo as applied to <option selected> (per CSS3 spec). Refs #5179

[bill]

Failing on IE6 too (unsurprisingly). Should either fix for 1.4 or punt this ticket and comment out the test until it's fixed.

[Sam Foster]

This seems to be a deficiency in QSA for webkit, and for IE8, so it will need some handling on that side. Probably the easiest is to add :checked to the list of indexOf checks which set useQSA:

			(query.indexOf(":contains") == -1) && query.indexOf(":checked") == -1)
...

then, the above change to test hasAttr("checked") and then hasAttr("selected") works in all cases.

The inline docs are a little self-contradictory on this. On one hand they say state selector are not supported, but then go on to say :checked is. If that's the case, this should be fixed. I'll check in the change if I get the nod, the unit test is already in place.

[James Burke]

(In [20843]) Fixes #5179, thanks to sfoster for doing the work figuring it out. Fixes :checked for option elements \!strict