Comments on snippet: 'Replace newlines with BR (platform safe)'

Emmanuel said on 11/19/10

Fri, 19 Nov 2010 18:44:43 GMT

Or try `str.replace(/([^>\%0D\%0A]?)(\%0D\%0A|%0A\%0D|\%0D|%0A)/g, '$1$2');`

muggo said on 9/2/10

Thu, 02 Sep 2010 15:18:45 GMT

Um. I take that back. This site strips html and reformats code... pretty silly for a code snippet site. My bad, I apologise.

muggo said on 9/2/10

Thu, 02 Sep 2010 15:17:15 GMT

Not only is this really ugly code it doesn't work. nickwicks addition doesn't either. This isn't much prettier but at least it works function nl2br (str) { var breakTag = ''; return (str + '').replace(/([^>\r\n]?)(\r\n|\n\r|\r|\n)/g, '$1'+ breakTag +'$2'); }

nickwick76 said on 8/16/09

Sun, 16 Aug 2009 16:57:13 GMT

When the text doesn't contain a newline of any kind, variable 're_nlchar' is not defined and an error is raised when returning from the function. A suggestion would be to replace the return statement with the following: if (typeof(re_nlchar) == "undefined") { return unescape( text ); } else { return unescape( text.replace(re_nlchar,'') ); }

Roshambo said on 8/2/06

Wed, 02 Aug 2006 00:00:00 GMT

The ironic thing about this post is that it itself could use some newline replacement.

cochambre said on 8/2/06

Wed, 02 Aug 2006 00:00:00 GMT

Tried that. Snipplr just didn\'t care whatever I typed. See the \\r\\n ?

cochambre said on 8/2/06

Wed, 02 Aug 2006 00:00:00 GMT

Something has broken with this site.

tylerhall said on 8/2/06

Wed, 02 Aug 2006 00:00:00 GMT

Yep, during last night's update I introduced a bug that was messing with newlines. It's been fixed. Sorry about that :)

tylerhall said on 8/2/06

Wed, 02 Aug 2006 00:00:00 GMT

I've gone through all of the affected snippets and corrected them.