search

googlearchive / core-header-panel

Simple panel with top and bottom panes
9 stars 8 forks source link

Fixes #27 #28

Closed ebidel closed 9 years ago

ebidel commented 9 years ago

R: @frankiefu

ebidel commented 9 years ago

cc @ssorallen

ssorallen commented 9 years ago

Is there evidence that this is causing problems? I'm not opposed to the change if there's proof that this fixes an issue.

ebidel commented 9 years ago

PTAL. The evidence is at https://github.com/Polymer/core-scroll-header-panel/issues/37, the but issue occurs on this element too. Entire page repaints are no good, and we should eliminate them. We discovered this in a demo app someone is building.

ssorallen commented 9 years ago

K sounds good to me. I had one more comment, but otherwise :+1:.