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

<polymer-ui-accordion> doesn't always work with Shadow DOM polyfill #444

Closed
esprehn opened this issue Mar 10, 2014 · 2 comments
Closed

<polymer-ui-accordion> doesn't always work with Shadow DOM polyfill #444

esprehn opened this issue Mar 10, 2014 · 2 comments
Assignees

Comments

@esprehn
Copy link

esprehn commented Mar 10, 2014

I have a template repeat over inside an accordion and I attempt to select the last one. The last one does get the active attribute, but it doesn't actually expand.

<polymer-ui-accordion multi selected="{{ [ messages.length - 1 ] }}">
    <template repeat="{{ message, i in messages }}">
        <polymer-ui-collapsible class="message {{ {approval: message.approval, disapproval: message.disapproval} | tokenList }}">

which produces this on the last row:

<polymer-ui-collapsible class="message polymer-selected" active="">
    <div on-tap="{{ headerTap }}">
      <div class="message-header polymer-ui-collapsible-header" on-tap="{{ scrollMessageToVisibleIfNeeded }}">
                     ...
    </div>
    <div id="collapsibleBody" on-tap="{{ bodyTap }}" class="polymer-collapse-closed" style="overflow: hidden;">
  ...

so it seems the <polymer-ui-collapsible> is confused as it has the active attribute set, the polymer-selected class added, but the body has polymer-collapse-closed and overflow: hidden.

If you click the header to collapse it and then expand it again the widget starts working.

@arv arv self-assigned this Mar 10, 2014
@frankiefu
Copy link
Member

In looking for the root cause, found 2 issues in ShadowDOM polyfill:
https://github.com/Polymer/ShadowDOM/issues/399
https://github.com/Polymer/ShadowDOM/issues/400

@tjsavage
Copy link
Contributor

Since polymer-ui-accordion is deprecated now, closing this here. Feel free to reopen

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

No branches or pull requests

4 participants