-
-
Notifications
You must be signed in to change notification settings - Fork 1k
Conversation
Β Open in CodeSandbox Web Editor | VS Code | VS Code Insiders |
β Deploy Preview for nuxt3-docs ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. /cc @atinux
Thanks @danielroe Shall we simply disable transitions by default in Nuxt 3? π€ |
There is merit in that. They cause issues like nuxt/nuxt#13471 when many users may not be using them at all ... wdyt @pi0? |
It also brings Nuxt 3 as light as possible when starting |
Makese sense to disable transitions by default. We only need proper way to guide users to enable it back |
I think disabling them by default is a good idea to reduce friction, as long as docs cover how to use them (e.g. with an extra section). |
π Linked issue
hotfix to work around nuxt/nuxt#15239
β Type of change
π Description
I do not believe we are currently using transitions so we probably should disable these anyway. This also serves to seriously reduce a memory leak that we're looking into.
π Checklist