Bug Tracker

Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#10656 closed bug (wontfix)

.removeData( list) does not appear to work

Reported by: [email protected] Owned by: Rick Waldron
Priority: low Milestone: None
Component: data Version: 1.6.4
Keywords: Cc:
Blocked by: Blocking:

Description

passing a list to removeData is broken as follows (tested in IE 9 and current Chrome):

passing a list does not remove the data:

http://jsfiddle.net/mKPYN/2/

accessing the data before passing a list causes removeData to error out when passed a list:

http://jsfiddle.net/mKPYN/3/

And this happens whether the data is assigned via jquery or in the html:

http://jsfiddle.net/mKPYN/4/

Change History (9)

comment:1 Changed 10 years ago by Rick Waldron

Component: unfileddata
Owner: set to Rick Waldron
Status: newassigned

comment:2 Changed 10 years ago by Rick Waldron

Priority: undecidedlow
Resolution: wontfix
Status: assignedclosed

.removeData() will not remove the actual data-* attributes from the DOM element. To actually remove an attribute, you must use removeAttr()

comment:3 Changed 10 years ago by [email protected]

Please see the third fiddle from the original ticket: http://jsfiddle.net/mKPYN/4/

comment:4 Changed 10 years ago by [email protected]

For clarification: the third fiddle from the original ticket appears to work, but only because the script errors out before it gets to the line where it would update the DOM.

comment:5 Changed 10 years ago by Rick Waldron

Keywords: needsdocs added

jQuery will remove data properties that it has created with data() (these are not placed on the DOM element itself), but will not remove data-* attributes. The extent of jQuery's involvement with data-* attributes is a read-only relationship.

http://jsfiddle.net/rwaldron/Zc98U/2/

comment:6 Changed 10 years ago by dmethvin

The fiddle was using 1.6.4 and the api page is pretty clear that arrays are a 1.7 feature.

I have updated the .removeData() page to make it excruciatingly explicitly incredibly verbosely clear that we don't touch data- attributes with this api. There is a whole article that says more about the whole topic:

http://www.learningjquery.com/2011/09/using-jquerys-data-apis

comment:7 Changed 10 years ago by [email protected]

Thank you, I did overlook the "version added: 1.7" on the api documentation, I didn't realize the documentation included unreleased features.

comment:8 Changed 10 years ago by dmethvin

Well jQuery 1.7 is now shipping, as of 10 minutes ago. The docs were updated yesterday. If we don't update the docs before we ship then people won't know what is new. It's hard to do it all simultaneously.

comment:9 Changed 10 years ago by addyosmani

Keywords: needsdocs removed
Note: See TracTickets for help on using tickets.