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

motion prop doesn't work, but the deprecated 'animation' prop does. However, it only accepts 'zoom.' #254

Closed
RayBans2111 opened this issue Apr 28, 2021 · 4 comments

Comments

@RayBans2111
Copy link

Hello,

I am using this component in order to render tooltips for my site. However, when I want to add an animation to the tooltip, no other animation name works other than 'zoom'. I tried playing around with the motion prop (Which isn't explained in the docs), and transitionName, but neither of those two work.

The zoom animation is a little too playful for my site, and I would just like to know how I can set a simple 'fade' animation instead. I tried looking into all the animation names, but the links I would visit were all dead ends.

It would be great to know the proper way to setup an animation for the tooltip as I can't find a solid place that tells me how to.

@Johnrobmiller
Copy link

Johnrobmiller commented May 29, 2022

Using transitionName as a prop is questionable. It burdens the dev by forcing them to manage the complexity of "transitionName". To make matters worse, the documentation is unhelpful and unclear -- it basically just says "Just look it up. you're on your own, kid..."

@sgehrman
Copy link

yeah, where are the docs for the normal fade in?

@RayBans2111
Copy link
Author

yeah, where are the docs for the normal fade in?

I was never able to figure out a solution for this issue, and I ended up not using this library in the end, unfortunately.

@sambolias
Copy link

This does not seem like it should be closed. It seems like it would be possible to define the motion prop using https://reactjs.org/docs/animation.html#high-level-api-reactcsstransitiongroup, but that isn't very convenient. Is this something in the works, that will be documented when complete?
I did find that transitionName does work when set to rc-tooltip-zoom, which I assume is because some CSSTransitionGroups are defined in rc-tooltip. It would be convenient to have some presets set up like that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants