-
Notifications
You must be signed in to change notification settings - Fork 67
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
fix(hmr): more precise matching #141
base: main
Are you sure you want to change the base?
Conversation
I've applied your patch with pnpm patch. When I edit the layout or one of its inner components it seems to trigger something, but the data in the browser is not updated. Is there anything special that I need to do to get this working? |
Any update here? |
1 similar comment
Any update here? |
Hello guys, @markthree, there is any possibility to use your specific branch in my project until PR is not accepted? Thanks in advance, |
@nos86 Sorry bro, no progress on this pr at the moment, you can try using vite-plugin-vue-meta-layouts |
any updates? |
In the latest vite demo test, if I manually update an already existing layout, restart is required for it to take effect, this pr fixes the issue。
There are no break changes and it works fine 🥰
A repo that reproduces the problem → vue-layout-demo
Start the service and modify
src/_base/layouts/Layout.vue
to find the problem.