String.clean

[JanK]

String.clean replaces tabs and newlines with a space, but only if theres more than one consecutive whitespace, better would be to replace all whitespace by one space.

"foo\t\tbar".clean(); // "foo bar"
"foo\tbar".clean(); // "foo\tbar"

The regexp /\s+/g could replace the current regexp.

[kamicane]

fixed in [1342]

[daniel@…]

It looks like IE has some serious problems with /s regex. IE ignores nbsp; and that is really strange, because another, a bit more descriptive notation, works well.

Test page:
http://daniel.steigerwald.cz/strangewhitespaceregex/

Please compare results in Firefox and Internet Explorer 7

I suggest to use this: [ \f\n\r\t\v\u00A0\u2028\u2029]+/g

[anonymous]

ECMAScript (PDF, see 1 5 .10.2.10) specifies \s as [\t\n\v\f\r], Firefox added [\u00A0\u2028\u2029] to the list if the docs are complete.

Opera happens to match   with \s like Firefox. Safari behaves like the IE and doesnt match &nbsp with \s.

I didnt test \u2028 and \u2029, but I think /[\s\u00A0\u2028\u2029]+/g should fix this for all browsers, as it adds firefox`s additions to \s.

[JanK]

that was me, JanK

[anonymous]

could be really useful links:

http://blog.stevenlevithan.com/archives/javascript-regex-and-unicode
http://blog.stevenlevithan.com/archives/faster-trim-javascript

sorry, i have no time to suggest code :(

[DanielSteigerwald]

that was me, Daniel Steigerwald