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

Fixes #3530. When updateStyles is called and an element is not atta… #3724

Merged
merged 3 commits into from
Jun 23, 2016

Conversation

sorvell
Copy link
Contributor

@sorvell sorvell commented Jun 18, 2016

Fixes #3530.

…ched, invalidate its styling so that when it is attached, its custom properties will be updated.

Also fixed a related issue with scope style cache disablement when :host function rules are used:

  • "property prepping" and decoration are now after static shimming so that transformed selectors are available to calculate :host function,
  • the detection of :host function properly accounts for type extension,
  • since "property prepping" was moved after shimming, apply shim was separated from this and moved before shimming so that its output can be used there.

…ched, invalidate its styling so that when it is attached, its custom properties will be updated.

Also fixed a related issue with scope style cache disablement when :host function rules are used:
 * "property prepping" and decoration are now after static shimming so that transformed selectors are available to calculate :host function,
 * the detection of :host function properly accounts for type extension,
 * since "property prepping" was moved after shimming, apply shim was separated from this and moved before shimming so that its output can be used there.
@dfreedm
Copy link
Member

dfreedm commented Jun 21, 2016

LGTM!

@dfreedm
Copy link
Member

dfreedm commented Jun 23, 2016

LGTM, #3708 is what broke the tests.

@dfreedm dfreedm merged commit 4b010c5 into master Jun 23, 2016
@dfreedm dfreedm deleted the fix-3530 branch June 23, 2016 03:29
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.

Calling updateStyles before an element is attached does nothing and is confusing
4 participants