trim method doesn't trim all white-space

[dotnetCarpenter]

The trim method only removes white-space from the beginning and end of a string. I often need to remove all space, e.i. a telephone number ect. Therefore I suggest a second optional boolean argument called 'all'. If 'all' is true, all white-space is removed.

Test Case var withSpace = $.trim(' 432 334 532 '); var noSpace = $.trim(' 432 334 532 ', true);

Patch /

• Uses a regular expression to remove whitespace from the given string.
• @param {String} text The string to trim.
• @param {Boolean} [all] True to remove all white-space from the text.
• @return {String} */

trim: function(text, all) {

return all===true ? (text

"").replace( /\s+/g, "") : (text

"").replace( /\s+|\s+$/g, "" );

}

[dotnetCarpenter]

Patch code without wiki formatting:

/**
 * Uses a regular expression to remove whitespace from the given string.
 * @param {String} text The string to trim.
 * @param {Boolean} [all] True to remove all white-space from the text.
 * @return {String}
 */
trim: function(text, all) {
  return all===true ? (text || "").replace( /\s+/g, "") : (text || "").replace( /^\s+|\s+$/g, "" );
}

[dotnetCarpenter]

Test Case

console.log($.trim(' 432 334 532 '));
console.log($.trim(' 432 334 532 ', true));

[aakoch]

I don't think you understand the meaning of "trim". I would just override trim with your function. I don't think this should be a part of jQuery.

[john]

This isn't something that we're going to be landing - should make for a fine plugin, though.