-
-
Notifications
You must be signed in to change notification settings - Fork 407
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
[Bug]: Background entry animation ignored #697
Comments
OK, found the issue! cookieconsent/src/core/modals/consentModal.js Line 282 in c4d512f
.cc--anim class is correctly added after a timeout of 100ms, while on show there is no such timeout cause the class is never removed.
See #698 for the fix |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Expected Behavior
The background of
. cm-wrapper
should animateCurrent Behavior
The background of
. cm-wrapper
does not animateSteps to reproduce
Hi and thank you for this awesome module! I found a strange bug regarding the background of the main consent window. You can test this even on playground.
CookieConsent.show()
(with no argument or true/false, doesn't matter)From a first sight I didn't catch anything wrong in the code, but I will check further later, do you already have any ideas maybe?
Proposed fix or additional info.
No response
Version
3.0.1
On which browser do you see the issue?
Chrome/Firefox
The text was updated successfully, but these errors were encountered: