[patch] [cla] Add "required" validation to CheckedMultiSelect

[Nicholas VanderPyle]

It would be nice if CheckedMultiSelect? had an option to require that at least one item be selected and tie into Dojo's built-in validation.

I've attached a patch that supports this on my local build.

The patch also adds a callback (OnAfterAddOptionItem?) when an item is added to the list.

[bill]

Hi Nicholas,

Actually can you file a ​CLA? We can't look at patches without them.

[Nicholas VanderPyle]

Hi Bill, I should have a CLA on file.

I'm listed here: ​http://www.dojotoolkit.org/dojo-contributors as: VanderPlye?, Nicholas

It should be "VanderPyle?". Who should I email to get that corrected?

Also, to prevent future confusion, how do I specify on my Trac login that I have a CLA?

[bill]

Ah great.

I didn't notice because of the typo. I think you can mail cla@… to get it fixed.

About the bugdb login: I don't know any way to mark users who have a CLA but if you write [patch] [cla] in the ticket summary (ie, the bug title) then we'll know.

[dante]

(In [22514]) fixes #10406 - adds validation support to multicheckbox, defaults to false. cleans up use of attr() throughout test and module, all tests pass. Also makes claro default theme and uses _testCommon