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

vue版本useAutoAnimate允许传入el,兼容旧版 #159

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

17359898647
Copy link

No description provided.

@vercel
Copy link

vercel bot commented Sep 1, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
auto-animate ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 6, 2023 8:51am

@justin-schroeder
Copy link
Member

Sorry, not sure what this PR improves exactly — can you help clear that up for me?

@17359898647
Copy link
Author

Sorry, not sure what this PR improves exactly — can you help clear that up for me?

Sorry for my poor English, the following passage is from Google Translate。
If you want to use the ref variable returned by your function to bind the vue component, my modification allows the user to pass in the ref variable of the bound component while retaining the original usage method.

@justin-schroeder
Copy link
Member

Sorry still not following what API this enabled. Could you share a code sample of what the desired API is?

@17359898647
Copy link
Author

Sorry still not following what API this enabled. Could you share a code sample of what the desired API is?

I submitted a sample code that can be run directly. I just added a new parameter based on the original one to implement a new usage method without any impact on the original usage method.

@17359898647
Copy link
Author

Now ref can not only bind native html tags, but also bind custom components of vue.

@17359898647
Copy link
Author

17359898647 commented Sep 12, 2023

Can it be allowed

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

Successfully merging this pull request may close these issues.

2 participants