Akryum / floating-vue

💬 Easy tooltips, popovers, dropdown, menus... for Vue
https://floating-vue.starpad.dev/
MIT License
3.26k stars 333 forks source link

Some changes in the docs for advanced transitions #1023

Open mschinwald opened 5 months ago

mschinwald commented 5 months ago

Hi 👋

The classes for the hiding transition are wrong in the table in https://floating-vue.starpad.dev/guide/css#advanced-transitions, so I changed them accordingly.

Also, I realized that if you follow the docs and change the transition of .v-popper__wrapper, you also have to change the transition of its parent (.v-popper__popper). So if your transition takes .5 seconds for example, you wouldn't be able to see the whole transition because of the parent's transition that takes only .15 seconds.

You can see an example of this in this stackblitz: https://stackblitz.com/edit/vue-fyqfc2?file=src%2FApp.vue,src%2Fmain.js

So I added a little note about it. I'm not sure if I explained it well and if it needs to be in the docs, but it probably would have saved me some time.

netlify[bot] commented 5 months ago

Deploy Preview for v-tooltip ready!

Name Link
Latest commit fd62511d64c4c53e197e07ada1cd32dae9373239
Latest deploy log https://app.netlify.com/sites/v-tooltip/deploys/65bc28f7c6e2250007268c54
Deploy Preview https://deploy-preview-1023--v-tooltip.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.