Skip to content

Fix issue (#160) where STYLE elements would not be updated properly in IE < 9 #263

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

dbergey
Copy link
Contributor

@dbergey dbergey commented May 11, 2011

Looks like IE doesn't allow updating of a STYLE tag's innerHTML before version 9. Updating the tag's stylesheet property instead, if it exists, falling back to innerHTML if it does not.

@lukeapage
Copy link
Member

looks like this was closed with a different pull request?

@matthew-dean
Copy link
Member

I think regardless, pull requests that cannot be merged should be closed and a second pull request made. Unless there's a better "Github" way to handle un-mergable requests.

strk pushed a commit to CartoDB/carto that referenced this pull request Mar 22, 2013
strk pushed a commit to CartoDB/carto that referenced this pull request Mar 22, 2013
stefanklug pushed a commit to stefanklug/carto that referenced this pull request May 21, 2013
stefanklug pushed a commit to stefanklug/carto that referenced this pull request May 21, 2013
Quote ampersend chars in XML text. Closes less#263.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants