Ticket #3562 (closed bug: fixed)

Opened 15 months ago

Last modified 15 months ago

.removeAttr() needs to support ARIA properties

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

Description

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

This only needs to support states, not roles.

Attachments

removeAttr.patch (1.2 kB) - added by scott.gonzalez 15 months ago.

Change History

Changed 15 months ago by scott.gonzalez

Changed 15 months ago by scott.gonzalez

  • 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.

Changed 15 months ago by scott.gonzalez

  • status changed from accepted to closed
  • resolution set to fixed

Fixed in r941.

Changed 15 months ago by scott.gonzalez

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