Bug Tracker

Ticket #3786 (closed bug: fixed)

Opened 6 years ago

Last modified 2 years ago

removeAttr should use jQuery.props

Reported by: scott.gonzalez Owned by: aflesler
Priority: high Milestone: 1.6
Component: attributes Version: 1.5.2
Keywords: attrhooks Cc:
Blocking: #8457 Blocked by:

Description

removeAttr sets the value to empty string using attr, which uses jQuery.props, but it also calls removeAttribute on the node, which doesn't use jQuery.props.

Change History

comment:1 Changed 6 years ago by dmethvin

  • Owner set to aflesler

comment:2 Changed 4 years ago by dmethvin

  • Component changed from core to attributes

Still applies in 1.4.3. Looking at jQuery.props I am not sure that all of them should be applied to removeAttribute. Definitely requires some testing.

comment:3 Changed 4 years ago by SlexAxton

  • Keywords attr removeAttr added
  • Priority changed from major to high
  • Status changed from new to assigned
  • Milestone changed from 1.3 to 1.4.5

Reassigning back to where it was before.

comment:4 Changed 4 years ago by dmethvin

  • Keywords attrhooks added; attr removeAttr removed

Note that this may affect the attrhooks rewrite.

comment:5 Changed 3 years ago by john

  • Blocking 8457 added

comment:6 Changed 3 years ago by timmywil

  • Status changed from assigned to closed
  • Version changed from 1.2.6 to 1.5.2
  • Resolution set to fixed
  • Milestone changed from 1.next to 1.6
Note: See TracTickets for help on using tickets.