-
Notifications
You must be signed in to change notification settings - Fork 156
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
feat(server-routes): implement persisting input values in localStorage #545
Conversation
Deploying with Cloudflare Pages
|
Shall we also remember the active tab for each route? |
@atinux For sure! |
Looks good to me! I let @antfu do the final review |
@@ -308,6 +308,50 @@ watchEffect(() => { | |||
} | |||
}) | |||
|
|||
const savedRouteInputs = useLocalStorage<{ path: string, tab: string, inputs: any }[]>('nuxt-devtools:server-routes:inputs', () => [], { | |||
window: window.parent, |
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.
Is there a reason to pass the parent window here? It should work with iframe window as they are same origin.
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.
While trying out with my laptop(windows 10), the storage wasn't working as expected(the client would get reset after a page refresh)
Is it possible for you to test it out?
Screen.Recording.2023-12-16.at.9.10.16.PM.mov