Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#3562 closed bug (fixed)

.removeAttr() needs to support ARIA properties

Reported by: Scott González Owned by: Scott González
Priority: major Milestone: 1.7
Component: [meta] ui.a11y Version: 1.6rc2
Keywords: Cc:
Blocked by: Blocking:

Description

.attr() is proxied for ARIA properties in FF2, but .removeAttr() isn't.

This only needs to support states, not roles.

Attachments (1)

removeAttr.patch (1.2 KB) - added by Scott González 10 years ago.

Download all attachments as: .zip

Change History (4)

Changed 10 years ago by Scott González

Attachment: removeAttr.patch added

comment:1 Changed 10 years ago by Scott González

Status: newaccepted

The attached patch assumes that if we're removing any ARIA property, that it's safe to just call removeAttributeNS, without clearing the value first or checking the node type. I'm not sure if that's correct.

comment:2 Changed 10 years ago by Scott González

Resolution: fixed
Status: acceptedclosed

Fixed in r941.

comment:3 Changed 10 years ago by Scott González

Milestone: TBD1.6
Note: See TracTickets for help on using tickets.