Bug Tracker

Opened 12 years ago

Closed 12 years ago

Last modified 11 years ago

#9129 closed bug (fixed)

jQuery does not support enumerated attributes such as contenteditable

Reported by: Timmy Willison Owned by: Timmy Willison
Priority: blocker Milestone: 1.6.1
Component: attributes Version: 1.6
Keywords: Cc:
Blocked by: Blocking:



I see no way around creating a regex of boolean attributes which are boolean by spec so that we do not interfere with enumerated attributes. The properties for enumerated attributes should actually be "true", "false", and "" (instead of same name, existent, empty string, and removed as in boolean attributes) and we can make this easy for our users with a regex.

The regex is also required when setting html5 boolean attributes, where those corresponding properties are undefined in browsers that do not support html5, so the boolean property check fails (for instance autofocus).

#9123 is related.

I think this will make things much simpler and true to spec, and actually be faster (once the regex is built).

Change History (3)

comment:1 Changed 12 years ago by Timmy Willison

Component: unfiledattributes
Milestone: 1.next1.6.1
Owner: set to Timmy Willison
Priority: undecidedblocker
Status: newassigned

comment:2 Changed 12 years ago by Timmy Willison

Resolution: fixed
Status: assignedclosed

Construct boolHook and rboolean regex to deal with boolean attributes. Fixes #9129. Fixes #9123.

  • Construct regex for all attributes which are boolean attributes by specification.

+ This allows us to do what users expect with enumerated attributes and allows us to no longer complicate the issue.

+ People are just too confused with passing "true" instead of true when they should, so we won't have to worry about that anymore.

comment:2 Changed 12 years ago by Timmy Willison

#9105 is a duplicate of this ticket.

Note: See TracTickets for help on using tickets.