Bug Tracker

#13401 closed bug (fixed)

Opened February 06, 2013 01:53PM UTC

Closed February 08, 2013 02:50PM UTC

Last modified May 24, 2013 01:43PM UTC

.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/

Attachments (0)
Change History (9)

Changed February 06, 2013 02:12PM UTC by markraddatz comment:1

Pull request at https://github.com/jquery/jquery/pull/1162

Changed February 07, 2013 01:18AM UTC by gibson042 comment:2

component: unfiled → manipulation
status: new → open
version: git → 1.9.0

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

Changed February 08, 2013 02:50PM UTC by Mark Raddatz comment:3

resolution: → fixed
status: open → closed

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

Changeset: 6a0ee2d9ed34b81d4ad0662423bf815a3110990f

Changed February 08, 2013 02:55PM UTC by Mark Raddatz comment:4

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

(cherry picked from commit 6a0ee2d9ed34b81d4ad0662423bf815a3110990f)

Changeset: 03ab9b9f9061df1cf38fd6466c2241dfb2ae9cdc

Changed March 20, 2013 04:54PM UTC by anonymous comment:5

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

Changed March 20, 2013 05:15PM UTC by dmethvin comment:6

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.

Changed March 20, 2013 05:25PM UTC by scottgonzalez comment:7

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?

Changed March 20, 2013 05:29PM UTC by dmethvin comment:8

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.

Changed May 24, 2013 01:43PM UTC by dmethvin comment:9

milestone: 1.9.2 → 1.10/2.0

Bulk update to milestone 1.10/2.0