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

Observers not called on component inside an iron-list template #4601

Closed
3 tasks done
fabysdev opened this issue May 16, 2017 · 3 comments
Closed
3 tasks done

Observers not called on component inside an iron-list template #4601

fabysdev opened this issue May 16, 2017 · 3 comments
Assignees
Labels

Comments

@fabysdev
Copy link

fabysdev commented May 16, 2017

Description

It seems like property observers are not called if the component is inside an iron-list template.
A dom-repeat renders fine outside of an iron-list but not inside of it.
If I call render on the dom-repeat it renders normally.

Live Demo

http://jsbin.com/pulaworude/edit?html,js,output

Expected Results

dom-repeat renders items

Actual Results

dom-repeat does not render items

Browsers Affected

  • Chrome
  • Firefox
  • Edge

Versions

  • Polymer: v2.0.0
  • webcomponents: v1.0.0

It works with version 2.0.0-rc.8. First broken version is 2.0.0-rc.9.

@kownacki
Copy link

I also have exactly the same problem. Thank you reporting it and finding workaround to use 2.0.0-rc.8.

@sorvell
Copy link
Contributor

sorvell commented May 17, 2017

Thanks for reporting this. We can confirm this issue. There's a simple fix and it should make the first bug release version.

The issue is in PropertyEffects at _readyClients. If a client has enabled properties prior to an element readying its clients (this can happen when using templatize), the host element should call _flushProperties on it.

@sorvell sorvell self-assigned this May 17, 2017
sorvell pushed a commit that referenced this issue May 18, 2017
… properties. This can happen when templatize is used to create instances with no properties. In this case, in order for properties to flush properly to clients, clients must be flushed.
@jogibear9988
Copy link

see #4609

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

No branches or pull requests

4 participants