-
-
Notifications
You must be signed in to change notification settings - Fork 109
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
The Lifecycle Hooks is out of order after v3.9.0 #697
Comments
After my testing, I found that it happened in other versions after 3.9.0 |
Hi @jackywongmm please add a minimal reproduction link? https://antfu.me/posts/why-reproductions-are-required |
https://stackblitz.com/edit/stackblitz-starters-uyd9qn?file=src%2Fviews%2FnormalView.vue |
When I tested bandicam.2024-06-05.21-43-46-771.mp4 |
It might be on how we are removing nodes on the custom renderer https://github.com/Tresjs/tres/blob/0d2767f3b832385e59926bc38b8e5d7dbabb1e4a/src/core/nodeOps.ts#L134C1-L174C4 but not entirely sure @andretchen0 what are your thoughts 💭? |
Off the cuff, I don't think there's anything in the current For me, the route is a complicating factor. Does the problem here manifest without a route change? E.g., with a simple In passing, I tried, but can't currently run this: https://stackblitz.com/~/github.com/hexianWeb/tresjs-playground |
Sorry for the late response, I built another simple |
MAYBE related to #796 ? |
Hi @hexianWeb could you check if it's fixed in the latest version of the package, the fix of #796 might solve the issue |
Describe the bug
It will also appear in other versions
Reproduction
N
Steps to reproduce
No response
System Info
No response
Used Package Manager
npm
Code of Conduct
The text was updated successfully, but these errors were encountered: