perf(RouterView): avoid parent rerenders when possible

[posva]

Close #1701

[netlify]

✅ Deploy Preview for vue-router canceled.

Name	Link
🔨 Latest commit	1191594
🔍 Latest deploy log	https://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 ...

[netlify]

✅ Deploy Preview for vue-router canceled.

Name	Link
🔨 Latest commit	b514499
🔍 Latest deploy log	https://app.netlify.com/sites/vue-router/deploys/6734899965314d0008004132

[pkg-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

[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.