Skip to content
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

Fix for IE. #1717

Merged
merged 1 commit into from
Jun 4, 2015
Merged

Fix for IE. #1717

merged 1 commit into from
Jun 4, 2015

Conversation

achimnol
Copy link
Contributor

@achimnol achimnol commented Jun 2, 2015

My website was not working on IE because the styles were missing -- and the IE javascript console said that it could not read "cssText" property. I dig down a little bit and found that this simple two-line patch makes it working perfectly on IE 11 as well as Chrome and Firefox. I haven't tested this on other versions of IE.

  • Ensure rule.propertyInfo is correctly set in style-properties.html. It was checking a wrong attribute: rule.properties instead of rule.propertyInfo before calling decorateRule().
  • Fix a condition check in custom-style.html when rule.propertyInfo is null or undefined.

 * Ensure rule.propertyInfo is correctly set in style-properties.html.  It was checking a wrong attribute: rule.properties instead of rule.propertyInfo before calling decorateRule().

 * Fix a condition check in custom-style.html when propertyInfo is null or undefined.
@kevinpschaaf
Copy link
Member

@sorvell PTAL

@sorvell
Copy link
Contributor

sorvell commented Jun 4, 2015

LGTM... thx!

sorvell pushed a commit that referenced this pull request Jun 4, 2015
@sorvell sorvell merged commit 896a04b into Polymer:master Jun 4, 2015
@achimnol
Copy link
Contributor Author

achimnol commented Jun 5, 2015

Thanks! :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants