Doesn't work in IE8

[jdpopkin]

I think mailcheck crashes in IE8 because of a single call to String.trim. I know IE8 is annoying, but we're so close!

I have a long bus ride coming up this weekend, so I'll probably take a shot at it.

[weilu]

How about adding a shim to add String.trim support to IE8 at the application level rather than trying to handle it at the library level?

[vitalvital]

Since it is a jQuery plugin - $.trim() should be used.

[weilu]

This library can be used as a jQuery plugin; it can also be used without jQuery. It's unfortunate that some devs still have to support IE8. If one has to do it, like I recommended above, the issue can be solved by pulling in a shim. Given how long ES5 has been around, it's just not a concern at the library level to support legacy browsers.

[jdpopkin]

Full compliance with IE8 would require that we replace var parts = email.trim().split('@'); with var parts = email.replace(/^[\s\uFEFF\xA0]+|[\s\uFEFF\xA0]+$/g, "").split('@'); as per Mozilla's shim for trim. I suppose it makes sense to handle this at an application level (as a lot of other JavaScript libraries seem to do).