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.
var style = document.createElement('style');
var useCssText = (style.styleSheet) ? true : false;
style.type = 'text/css';
results in useCssText = false in IE7 which is causing wrong logic to execute for attaching css styles.
var style = document.createElement('style');
style.type = 'text/css';
var useCssText = (style.styleSheet) ? true : false;
results in useCssText = true, hence moving up style.type assignment.
IE7 also throws 'Access denied' exception on
style.styleSheet.cssText = [style.styleSheet.cssText, text].join('\n');
It can not read value of the
style.styleSheet.cssText
, so replacing it withstyle.innerHTML
.I've checked value of
style.innerHTML
before and after adding styles usingstyle.styleSheet.cssText
. InnerHTML correctly reflects changes made via cssText so I think it should be a safe change.