Bug Tracker

Opened 6 years ago

Closed 6 years ago

Last modified 5 years ago

#13401 closed bug (fixed)

.replaceWith( "" ) with empty string not working

Reported by: markraddatz Owned by:
Priority: undecided Milestone: 1.10/2.0
Component: manipulation Version: 1.9.0
Keywords: Cc:
Blocked by: Blocking:

Description

Choose an element you want to replace with an empty string. 

$( "p" ).replaceWith( "" );

Expect the same behaviour as .remove(), but nothing happens. Works in version 1.8.3. Tested with version: >= 1.9.0, browser: Firefox 18.0.1, os: Ubuntu 12.04.

See: ​http://jsfiddle.net/TLFGM/

Change History (9)

comment:2 Changed 6 years ago by gibson042

Component: unfiled → manipulation
Status: new → open
Version: git → 1.9.0

Confirmed; .replaceWith was implemented without .domManip before 1.9.

comment:3 Changed 6 years ago by Mark Raddatz

Resolution: → fixed
Status: open → closed

Fix #13401: replaceWith(""). Close gh-1163.

Changeset: 6a0ee2d9ed34b81d4ad0662423bf815a3110990f

comment:4 Changed 6 years ago by Mark Raddatz

Fix #13401: replaceWith(""). Close gh-1163. (cherry picked from commit 6a0ee2d9ed34b81d4ad0662423bf815a3110990f)

Changeset: 03ab9b9f9061df1cf38fd6466c2241dfb2ae9cdc

comment:5 Changed 6 years ago by anonymous

This isn't fixed in 1.9.1. replaceWith("") still doesn't work.

comment:6 Changed 6 years ago by dmethvin

Well 1.9.1 was fixed Feb 4 and this was landed Feb 8. So it will be in the next release. Please look at the dates next time and you can figure that out.

comment:7 Changed 6 years ago by scottgonzalez

dmethvin: This is closed: fixed with a milestone of none. Shouldn't it be set to either 1.9.2, 1.10.0, or 2.0.0?

comment:8 Changed 6 years ago by dmethvin

Milestone: None → 1.9.2

I'd held off on putting this in a milestone because we don't know if 1.9.2 or 1.10 was next. I'll just slot it into 1.9.2 for now.

comment:9 Changed 5 years ago by dmethvin

Milestone: 1.9.2 → 1.10/2.0

Bulk update to milestone 1.10/2.0

Note: See TracTickets for help on using tickets.