Bug Tracker

Modify

Ticket #3786 (closed bug: fixed)

Opened 5 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 5 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 3 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 3 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

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.