Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#3562 closed bug (fixed)

.removeAttr() needs to support ARIA properties

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


.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 scottgonzalez 8 years ago.

Download all attachments as: .zip

Change History (4)

Changed 8 years ago by scottgonzalez

comment:1 Changed 8 years ago by scottgonzalez

  • Status changed from new to accepted

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 8 years ago by scottgonzalez

  • Resolution set to fixed
  • Status changed from accepted to closed

Fixed in r941.

comment:3 Changed 8 years ago by scottgonzalez

  • Milestone changed from TBD to 1.6
Note: See TracTickets for help on using tickets.