Skip to content
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

[Bug Report][3.7.0] List Item throws "RangeError: Maximum call stack size exceeded" when using to prop #20387

Closed
ldavisCEI opened this issue Aug 23, 2024 · 3 comments
Labels
S: triage upstream Problem with a third party library that we may have to work around

Comments

@ldavisCEI
Copy link

Environment

Vuetify Version: 3.7.0
Vue Version: 3.4.38
Browsers: Edge 127.0.0.0
OS: Windows 10

Steps to reproduce

  1. Click one of the list items shown on the HomeView
  2. Issue: it doesn't show the HelloWorld view but the URL has been updated
  3. If you open the dev tools you will see the error message in the Console tab

Expected Behavior

It should show the HelloWorld component.

Actual Behavior

It stays on the HomeView with the list items.

Reproduction Link

https://github.com/ldavisCEI/vuetify-list-item-to-prop-issue

@J-Sek
Copy link
Contributor

J-Sek commented Aug 25, 2024

Router links usually respond to URL changes by setting active class (so your navigation item looks different when it's address in to matches registered path). I noticed computed value get's called a lot there.

image

Even if you think having 1000 links with the same route is somehow a valid use-case, it seems you should rather focus on Vue Router or Vue itself if you want this problem eliminated.

Or just replace to= with href=. The only downside will be missing page transition.

@KaelWD
Copy link
Member

KaelWD commented Aug 25, 2024

Yeah this is all in vue and vue-router's code, I imagine you'd get the same thing if you had 1000 components with just useLink(props)

@ldavisCEI
Copy link
Author

Took a look at this again. The issue is fixed by updating to Vue 3.5.6. https://github.com/ldavisCEI/vuetify-list-item-to-prop-issue has be updated and works now.

@KaelWD KaelWD closed this as not planned Won't fix, can't repro, duplicate, stale Sep 17, 2024
@KaelWD KaelWD added the upstream Problem with a third party library that we may have to work around label Sep 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S: triage upstream Problem with a third party library that we may have to work around
Projects
None yet
Development

No branches or pull requests

3 participants