Conversation

posva

Close #1701

@netlifyNetlify

Deploy Preview for vue-router canceled.

NameLink
🔨 Latest commit1191594
🔍 Latest deploy loghttps://app.netlify.com/sites/vue-router/deploys/657c544dce062d0008af5cae

@codecov-commenter

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.29%. Comparing base (e978eb8) to head (b514499).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2082   +/-   ##
=======================================
  Coverage   97.28%   97.29%           
=======================================
  Files          26       26           
  Lines        6077     6090   +13     
  Branches      836      836           
=======================================
+ Hits         5912     5925   +13     
  Misses        162      162           
  Partials        3        3           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Narretz

Hi @posva is this still relevant? Perf fixes are always good and I wonder if this fixes a bug I'm seeing ...

@netlifyNetlify

Deploy Preview for vue-router canceled.

NameLink
🔨 Latest commitb514499
🔍 Latest deploy loghttps://app.netlify.com/sites/vue-router/deploys/6734899965314d0008004132

@pkg-pr-newpkg.pr.new

Open in Stackblitz

pnpm add https://pkg.pr.new/vue-router@2082

commit: b514499

@posva

@Narretz can you try using the pkg-pr published version?

@Narretz

@posva my problem was actually solved in a vue 3.5+ update

@Narretz

Hi @posva this has actually become relevant (to me) again, and as you can see from #1701, devs are still experiencing this as a problem and reporting it. I tested my case with the MR build and it fixes it.. So can this be merged and released? I guess there's a chance that some apps actually rely on the re-render behavior by now, but it still feels like a bug/unexpected behavior to me.

@Narretz

Hi @posva can you give this another thought/look? See my comment above

@posvaposva moved this from 🆕 Triaging to 💬 In discussion in Vue Router Roadmap Mar 31, 2025
@posva

This still requires more testing in different scenarios with Keep Alive, Transition, and Suspense to ensure everything keeps working. I haven't had the time to test all the combinations yet. There are other pending comments in the named issue. I will eventually get back to it.

Sign up for free to join this conversation on . Already have an account? Sign in to comment
None yet
Status: 💬 In discussion

Successfully merging this pull request may close these issues.

View Component unnecessarily updates when child route changes