PolymerElements / paper-drawer-panel

A Material Design two-section responsive panel
https://www.webcomponents.org/element/PolymerElements/paper-drawer-panel
25 stars 42 forks source link

_computeMediaQuery never called on Firefox #100

Closed johnheroy closed 9 years ago

johnheroy commented 9 years ago

I have a <paper-drawer-panel> with a right drawer which works on Chrome (in shady DOM) but not in Firefox or iOS Safari. I found that the _computeMediaQuery method never gets called so the 'query' prop on is undefined and thus the parent <paper-drawer-panel> never gets a .narrow-layout class which triggers this style and actually slides away the drawer. Any thoughts?

johnheroy commented 9 years ago

forceNarrowChanged also never called, wonder if it has anything to do with https://github.com/PolymerElements/paper-drawer-panel/issues/26?

frankiefu commented 9 years ago

I can't reproduce that problem you mentioned. For example, this jsbin works on Firefox and Safari.

johnheroy commented 9 years ago

I am having trouble producing a simple example which I can share here and repros the problem, I created this jsbin which shows what we are doing (create paper-drawer-panel inside div and then attach div to the DOM). Btw I am using vulcanize and the lite version of the polyfill and compiling the components + deps with the closure compiler.

blasten commented 9 years ago

It could potentially be a timing issue in webcomponent.js. I have seem similar issues in the past; however, to effectively determinate the causes and its implication, I need to debug the code. Feel free to ping me in GH.