replace multiple tags with javascript [duplicate]

Question

I want to replace the multiple   tags with single   using JavaScript in a text. my text like:

some text.<br />
<br />
<br />
<br />
<br />
<br />
<br />
some text

I tried this code:

document.body.innerHTML = document.body.innerHTML.replace(/<br /> <br />
    <br /> <br /><br /> <br />/g,"<br />"); </script>

but it did not work .

I tried this code: document.body.innerHTML = document.body.innerHTML.replace(/ /g," "); </script> but it did not work — Ahmed Elshorbagy
– Ahmed Elshorbagy, Commented Jan 23, 2015 at 15:34
Do you want to replace occurences of " " in a string, or   elements in the DOM? — pawel
– pawel, Commented Jan 23, 2015 at 15:34
@AhmedElshorbagy: To improve your question, use the "edit" link, not comments. — T.J. Crowder
– T.J. Crowder, Commented Jan 23, 2015 at 15:34

pawel · Accepted Answer · 2015-01-23 15:48:03Z

7

You have specified that it's in document.body, so in the DOM. Then you can (and you should) use DOM methods:

var siblingBrs = [].slice.call( document.querySelectorAll('br + br') );
siblingBrs.forEach( function( br ){
    br.parentNode.removeChild( br );
});

http://jsfiddle.net/jL8jkkt0/

Notice it's agnostic to the string representation of a   element in the source code.

BTW if it's only for display purposes (to avoid visual gap) you can just use CSS:

br + br { display:none; }

http://jsfiddle.net/jL8jkkt0/1/

edited Jan 23, 2015 at 15:48

answered Jan 23, 2015 at 15:41

pawel

37.1k7 gold badges59 silver badges54 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

jrarama · Accepted Answer · 2015-01-23 15:54:35Z

1

First get the HTML as string then replace it using regex:

var s = document.body.innerHTML;
document.body.innerHTML = s.replace(/(\<br[\s]*\/?\>[\s]*)+/g, '<br/>');

First Line<br><br /><br/><br><br /><br/>Second Line

answered Jan 23, 2015 at 15:54

jrarama

9147 silver badges8 bronze badges

1 Comment

pawel Over a year ago

recommending regex to manipulate HTML is a poor advice stackoverflow.com/a/1732454/4879

Graham · Accepted Answer · 2015-01-23 15:36:21Z

0

Try the following:

var stripped = original.replace(/(<br\s*\/?>){3,}/gi, '<br>');

Substituting different values here...

{3,}

...will allow you to control at what count the removal occurs.

Hope this helps!

answered Jan 23, 2015 at 15:36

Graham

1621 silver badge16 bronze badges

Comments

Vijay · Accepted Answer · 2015-01-23 15:34:46Z

-1

Try this :

var newText = oldText.replace(/(<br\s*\/?>){3,}/gi, '<br>');

working fiddle:

http://jsfiddle.net/r5bu1xq2/

answered Jan 23, 2015 at 15:34

Vijay

3,0231 gold badge17 silver badges24 bronze badges

Collectives™ on Stack Overflow

replace multiple <br> tags with javascript [duplicate]

4 Answers 4

Comments

1 Comment

Comments

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

1 Comment

Comments

Comments

Linked

Related