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 scoping when class is not specified on element #3222

Merged
merged 1 commit into from
Feb 11, 2016

Conversation

nazar-pc
Copy link
Contributor

If class on element is not present then

(selector = element.getAttribute('class')) === null

And as result:

(selector + 'style-scope') === 'nullstyle-scope'

While just style-scope was expected.

nazar-pc added a commit to nazar-pc/CleverStyle-Framework that referenced this pull request Dec 20, 2015
@TimvdLippe
Copy link
Contributor

Could you add a little test to prevent regressions? Otherwise LGTM 👍

@nazar-pc nazar-pc force-pushed the fix-for-scoping-empty-class branch from 4e223b3 to 57f0df0 Compare February 1, 2016 00:48
nazar-pc added a commit to nazar-pc/CleverStyle-Framework that referenced this pull request Feb 1, 2016
@dfreedm
Copy link
Member

dfreedm commented Feb 4, 2016

Please add a test case that tests the problem this PR is fixing. See CONTRIBUTING.md for more details.

@nazar-pc nazar-pc force-pushed the fix-for-scoping-empty-class branch from 57f0df0 to 24e9fc7 Compare February 4, 2016 01:11
@nazar-pc
Copy link
Contributor Author

nazar-pc commented Feb 4, 2016

Test is here

@sorvell sorvell self-assigned this Feb 8, 2016
@sorvell
Copy link
Contributor

sorvell commented Feb 11, 2016

LGTM, thx!

sorvell pushed a commit that referenced this pull request Feb 11, 2016
Fix for scoping when class is not specified on element
@sorvell sorvell merged commit a4cc272 into Polymer:master Feb 11, 2016
@nazar-pc nazar-pc deleted the fix-for-scoping-empty-class branch February 11, 2016 20:46
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.

5 participants