Bug Tracker

Opened 8 years ago

Closed 5 years ago

#3270 closed bug (worksforme)

jQuery().wrapInner() prepends instead of wrapping when html structure contains text nodes

Reported by: jtate Owned by: flesler
Priority: major Milestone: 1.3
Component: core Version: 1.2.6
Keywords: Cc:
Blocked by: Blocking:

Description

For example:

jQuery('p').wrapInner('<strong/>');

would work fine. However

jQuery('p').wrapInner('<strong> </strong>')

will produce html output that looks like:

<p><strong> </strong>INNER TEXT</p>

when it meets a paragraph element with the Text contents of "INNER TEXT".

Change History (3)

comment:1 Changed 7 years ago by dmethvin

The problem is this loop in wrapAll:

	while ( elem.firstChild )
		elem = elem.firstChild;

It's trying to find the deepest point in the wrapping nodes to know where to append the nodes in the jQuery object. However, if any of the firstChild nodes in the wrapper are text nodes, it won't work right. I've attached a patch that tries to find the leftmost non-text node.

comment:2 Changed 7 years ago by dmethvin

  • Owner set to flesler

comment:3 Changed 5 years ago by dmethvin

  • Resolution set to worksforme
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.