Bug Tracker

Opened 8 years ago

Closed 6 years ago

Last modified 5 years ago

#3786 closed bug (fixed)

removeAttr should use jQuery.props

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

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 (6)

comment:1 Changed 8 years ago by dmethvin

Owner: set to aflesler

comment:2 Changed 7 years ago by dmethvin

Component: coreattributes

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 7 years ago by SlexAxton

Keywords: attr removeAttr added
Milestone: 1.31.4.5
Priority: majorhigh
Status: newassigned

Reassigning back to where it was before.

comment:4 Changed 7 years ago by dmethvin

Keywords: attrhooks added; attr removeAttr removed

Note that this may affect the attrhooks rewrite.

comment:5 Changed 6 years ago by john

Blocking: 8457 added

comment:6 Changed 6 years ago by timmywil

Milestone: 1.next1.6
Resolution: fixed
Status: assignedclosed
Version: 1.2.61.5.2
Note: See TracTickets for help on using tickets.