#3562 closed bug (fixed)

Opened November 12, 2008 03:00AM UTC

Closed November 15, 2008 04:29AM UTC

Last modified November 15, 2008 04:29AM UTC

.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:
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 scottgonzalez November 12, 2008 02:11PM UTC.
Change History (3)

Changed November 12, 2008 02:14PM UTC by scottgonzalez comment:1

status: new → 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 November 15, 2008 04:29AM UTC by scottgonzalez comment:2

resolution: → fixed
status: accepted → closed

Fixed in r941.

Changed November 15, 2008 04:29AM UTC by scottgonzalez comment:3

milestone: TBD → 1.6