Skip to content
This repository has been archived by the owner on Mar 13, 2018. It is now read-only.

Subsequent clicks cause the full menu to flash on screen before the reveal animation #45

Closed
chrisconover opened this issue Dec 19, 2014 · 15 comments

Comments

@chrisconover
Copy link

The paper-dropdown-menu reveal animation works correctly on the first click, but subsequent clicks cause the full menu to flash on screen before the reveal animation begins.

I've made a basic example at http://jsbin.com/lawuli

Here's a screencast demonstrating the issue as well: https://drive.google.com/file/d/0B8IK6wolRJizSEV0MmhXNWJFc0k/view?usp=sharing

@Ortham
Copy link

Ortham commented Dec 20, 2014

👍 for this, I think it was introduced in v0.5.2.

Edit: it only happens in Chrome (39). Firefox (34) and IE (11) both function as normal.

@morethanreal
Copy link

I think it's due to this bug: web-animations/web-animations-next#260

@alancutter
Copy link

The web-animations-js bug looks to be fixed in v1.0.5 according to web-animations/web-animations-next#260 however core-animation v0.5 is still using web-animations-js v1.0.1. Should its v0.5 bower be updated to point at web-animations-js#~1.0.5?

@morethanreal
Copy link

Updated in googlearchive/core-animation@ce0a08e

@richieb
Copy link

richieb commented Jan 30, 2015

This problem still exists, even after updating web-animations to 1.0.5 and core-animations to 0.5.4. Could someone confirm that it has indeed been fixed?

@pflannery
Copy link

This is working for me locally @ core-animations 0.5.4 and web-animations 1.0.5..but it's not working at https://www.polymer-project.org/components/paper-dropdown-menu/demo.html

@Ortham
Copy link

Ortham commented Jan 31, 2015

I'm still experiencing this with Polymer 0.5.4 (and web-animations 1.0.5) too.

@RobertLowe
Copy link

@morethanreal Can confirm as well. This is still happening on 0.5.4 & 1.0.5

Can we reopen the issue?

@Ortham
Copy link

Ortham commented Feb 8, 2015

After some more investigation, this bug only manifests under Chromium 40 and earlier. It's fixed in Chromium 41 (tested using the Chromium Embedded Framework branch 2272) and 42 (tested using the Chrome Canary).

@rjwright
Copy link

I'm wondering if this could be a result of this bug web-animations/web-animations-next#278 (which turned out to be a Chrome 40 issue, not a Web Animations Polyfill issue).

@morethanreal
Copy link

@rjwright Is there a workaround for Chrome 40?

@RobertLowe
Copy link

@morethanreal googlearchive/paper-dropdown#6 (comment) or googlearchive/paper-dropdown#6 (comment) insuring the state is immediately set or reverted worked for me.

As for a workaround I have nothing else to offer, cheers.

@morethanreal
Copy link

I'm resetting the dropdown background in googlearchive/paper-dropdown@62a37d5, which seems to fix this issue.

@RobertLowe
Copy link

@morethanreal you're awesome! 👍

@rjwright
Copy link

@morethanreal I will have a look Today to see if we can put a workaround in the polyfill.

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

No branches or pull requests

8 participants