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

Tweaked animations? #12

Open
elimisteve opened this issue Mar 31, 2020 · 8 comments
Open

Tweaked animations? #12

elimisteve opened this issue Mar 31, 2020 · 8 comments
Labels
help wanted Extra attention is needed

Comments

@elimisteve
Copy link

The incoming and outgoing default animation for https://fkhadra.github.io/react-toastify/ is so awesome. Would you consider tweaking the existing animation to support it? If not, would you accept a PR that swaps the current animation with the default react-toastify one?

Thanks!

@antony
Copy link
Member

antony commented Mar 31, 2020

Yep - I love it :)

I didn't write the animations, somebody else added them in a PR. They're decent, but I do prefer the toastify ones, so I'll be happy to take a PR.

@antony antony added the help wanted Extra attention is needed label Mar 31, 2020
@tmadge
Copy link

tmadge commented Sep 30, 2020

This broke things for me.... now the notification slides in and immediately disappears. I think its opacity flips back to 0 for some reason.

@antony
Copy link
Member

antony commented Sep 30, 2020

What broke things? There hasn't been any work done on this issue as far as I know.

@elimisteve
Copy link
Author

@tmadge is probably referring to https://github.com/beyonk-adventures/svelte-notifications/pull/17/files

@antony
Copy link
Member

antony commented Sep 30, 2020

It seems to work fine for me - https://svelte.dev/repl/dd506c546df84c1994a5ae9928ad23b1?version=3.12.1

Do you have any more detail? browser etc?

@elimisteve
Copy link
Author

@tmadge If you can make the notifications zip off to the right (like https://fkhadra.github.io/react-toastify/ ) rather than fading away then that'd be great! @timsully and I haven't gotten around to doing that part yet, but I sure think it'd be an improvement.

@tmadge
Copy link

tmadge commented Oct 1, 2020

Thanks for the repl. It must be something in my site that's causing it to hide.

I added opacity: 1 to the to section of the animate-in keyframe, which seems to have fixed it.

to {
  opacity: 1;
  transform: none;
}

@phelbas
Copy link

phelbas commented Dec 14, 2020

I had to make the same opacity patch because out of the box had the same issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants