Add polyfill for String.prototype.trim
#2908
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
In bc843a3 (#2840) we introduced a new
normaliseString
helper method which relies onString.prototype.trim
, which is not supported in IE8 or IE9.This causes a script error when going to any page in the review app (in IE8) which contains a component that accepts config via data attributes.
Add a polyfill for
String.prototype.trim
based on the polyfill that used to be on MDN (before they removed content relating to IE8), adapted to match the format used for the existing polyfills.Closes #2903.