fix: `<Suspense>` + `<Transition>` means mounted() runs too early #5952

liulinboyi · 2022-05-18T19:11:28Z

close #5844
fix: <Suspense> + <Transition> means mounted() runs too early

liulinboyi · 2022-05-18T19:16:41Z

Hugofgx9 · 2022-07-07T08:46:22Z

when will this be release ? This issue causes bugs on nuxt 3 which make it difficult to use in production for the moment

atinux · 2022-07-07T14:51:27Z

What is stopping this PR from changing its draft status?

richgcook · 2022-08-29T10:15:08Z

Any thoughts on when this fix would be merged?

richgcook · 2022-10-06T11:22:06Z

Any thoughts on when this fix would be merged?

Anyone?

warflash · 2022-10-06T11:45:21Z

🙏

BenjaminOddou · 2022-10-23T13:27:00Z

👀

stikoo · 2022-11-11T11:15:35Z

Oh, this would solve soooooo many issues we're having with Nuxt 3 right now, please please give this the love it deserves! 🙏

fabienbergerat · 2022-11-11T11:17:37Z

@stikoo agreed... For me this issue makes nuxt 3 pretty much pointless. It made me lose so much time trying to figure out the least ugly workarounds 🤯

Aljodomo · 2022-11-14T09:39:50Z

Great!

fabienbergerat · 2022-12-16T14:37:14Z

Is there any update on the status of this issue?

Willadaygo · 2023-04-26T11:53:45Z

Nudge nudge 🥲?

brunodeangelis · 2023-04-26T12:44:16Z

No disrespect to Vue maintainers, but I just find amazing that it's been a year since this PR was posted and there are no signs of merging yet. This bug heavily impacts the biggest and most used Vue framework, Nuxt 3, and makes it a huge pain to deal with what would be an otherwise excellent DX (and UX for that matter).

zsavajji · 2023-05-10T12:15:15Z

This single issue is preventing me from trying to migrate our next projects to the Nuxt 3 ecosystem, it's a pretty huge deal breaker 😢

maximilliangeorge · 2023-05-11T12:10:51Z

For anyone else waiting, this composable can work as an interim solution:
nuxt/nuxt#13471 (comment)

liulinboyi · 2023-05-11T15:28:37Z

For anyone else waiting, this composable can work as an interim solution: nuxt/nuxt#13471 (comment)

The workaround is Cool! PlayGround

nuxt/nuxt#13471 (comment)

davidwebca · 2023-05-31T17:54:33Z

We rely on this in our workflow, this seems to happen only when JS transitions are used and with mode "out-in" if I'm not wrong. Commenting to follow updates.

beejaz · 2023-07-09T10:55:04Z

Would be nice if this bug could be squeezed. Stuck in Nuxt3 workarounds for now.

michaelpumo · 2023-07-22T19:25:31Z

I can't believe it's been 1 year + 2 months since this PR was created and it's still not merged? This is a pretty major bug and the kind of bug that can break applications without realising it. Is this normal? It would be great if we can get this merged in and sorted once and for all. Thanks.

joseehilton147 · 2023-08-01T15:17:08Z

1 year and 3 months and counting, what's is stopping this from being merged?

JasonGotGithub · 2023-08-28T10:46:47Z

Out of curiosity, is this shelved or pending merging? From a creative development standpoint, it makes migrating to Vue 3 hard so it would great to get an update on it. Also, to assess if a search to an alternative solution (or framework) needs to happen in the case where the current behavior is the intended behavior

JasonGotGithub · 2023-09-14T09:38:42Z

Hi @sodatea, would you be able to shed some light on this?

haoqunjiang · 2023-09-27T08:03:39Z

This is a p4 issue, meaning we'll prioritize it when reviewing and merging the next batch of PRs.
But this PR, in its current status, isn't ready for merge (or even for review) because it lacks tests. Besides, the branch is far behind the current main branch. The code must be rebased first.
If you are really bothered by the bug, there's a workaround in the comments: onMounted hook running before DOM is available nuxt/nuxt#13471 (comment) Please give it a try.

maximilliangeorge · 2023-09-27T08:08:16Z

@sodatea Thanks for updating us 💖 I can vouch for the aforementioned workarounds, I'm currently using them successfully. But it will be great to see a proper solution.

stoner-w · 2023-09-27T08:35:26Z

@sodatea Dear Vue team members.

I would like to inquire if there is a plan for merging PRs? While I have been in the process of upgrading to Vue 3, I have encountered several issues, and some of these issues could be resolved through specific PRs. However, these PRs have remained in a state of suspension for some time now. Consequently, I have had to resort to alternative methods to address my problems, but I believe this is not an ideal approach. Moreover, these temporary solutions may potentially lead to issues in my application once these PRs are eventually fixed.

daniluk4000 · 2023-09-27T08:48:07Z

@sodatea Dear Vue team members.

I would like to inquire if there is a plan for merging PRs? While I have been in the process of upgrading to Vue 3, I have encountered several issues, and some of these issues could be resolved through specific PRs. However, these PRs have remained in a state of suspension for some time now. Consequently, I have had to resort to alternative methods to address my problems, but I believe this is not an ideal approach. Moreover, these temporary solutions may potentially lead to issues in my application once these PRs are eventually fixed.

I have pack of answers there:
#7050
#8560

stoner-w · 2023-09-27T09:01:11Z

@daniluk4000 I hope it can be processed as soon as possible, many issues have been bothering me for a long time.

daniluk4000 · 2023-09-27T09:03:13Z

@daniluk4000 I hope it can be processed as soon as possible, many issues have been bothering me for a long time.

Lol I didn't see you literally commented the last one, sorry.

I hope it can be processed as soon as possible

So do I.

pikax · 2023-10-20T06:29:03Z

Closing this in favour of #9388

close #5844 close #5952

fix: <Suspense> + <Transition> means mounted() runs too early

509fbb5

MarcRO3 approved these changes Nov 14, 2022

View reviewed changes

maximilliangeorge mentioned this pull request Apr 26, 2023

<Suspense> + <Transition> means mounted() runs too early #5844

Closed

maximilliangeorge mentioned this pull request Jun 6, 2023

Page transitions bad lifecycle ? nuxt/nuxt#15793

Closed

sxzz force-pushed the main branch from 39d4fc1 to 1b56405 Compare August 21, 2023 05:00

pikax mentioned this pull request Oct 12, 2023

fix(Suspense): mount runs too early when used in Transition #9388

Merged

fanckush mentioned this pull request Oct 16, 2023

onMounted hook running before DOM is available nuxt/nuxt#13471

Closed

pikax closed this Oct 20, 2023

yyx990803 pushed a commit that referenced this pull request Oct 21, 2023

fix(Suspense): calling hooks before the transition finishes (#9388)

00de3e6

close #5844 close #5952

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: `<Suspense>` + `<Transition>` means mounted() runs too early #5952

fix: `<Suspense>` + `<Transition>` means mounted() runs too early #5952

liulinboyi commented May 18, 2022

liulinboyi commented May 18, 2022

Hugofgx9 commented Jul 7, 2022 •

edited

Loading

atinux commented Jul 7, 2022

richgcook commented Aug 29, 2022

richgcook commented Oct 6, 2022

warflash commented Oct 6, 2022

BenjaminOddou commented Oct 23, 2022

stikoo commented Nov 11, 2022

fabienbergerat commented Nov 11, 2022

Aljodomo commented Nov 14, 2022

fabienbergerat commented Dec 16, 2022

Willadaygo commented Apr 26, 2023

brunodeangelis commented Apr 26, 2023

zsavajji commented May 10, 2023

maximilliangeorge commented May 11, 2023

liulinboyi commented May 11, 2023 •

edited

Loading

davidwebca commented May 31, 2023

beejaz commented Jul 9, 2023

michaelpumo commented Jul 22, 2023

joseehilton147 commented Aug 1, 2023

JasonGotGithub commented Aug 28, 2023 •

edited

Loading

JasonGotGithub commented Sep 14, 2023

haoqunjiang commented Sep 27, 2023

maximilliangeorge commented Sep 27, 2023 •

edited

Loading

stoner-w commented Sep 27, 2023

daniluk4000 commented Sep 27, 2023

stoner-w commented Sep 27, 2023

daniluk4000 commented Sep 27, 2023

pikax commented Oct 20, 2023

fix: <Suspense> + <Transition> means mounted() runs too early #5952

fix: <Suspense> + <Transition> means mounted() runs too early #5952

Conversation

liulinboyi commented May 18, 2022

liulinboyi commented May 18, 2022

Hugofgx9 commented Jul 7, 2022 • edited Loading

atinux commented Jul 7, 2022

richgcook commented Aug 29, 2022

richgcook commented Oct 6, 2022

warflash commented Oct 6, 2022

BenjaminOddou commented Oct 23, 2022

stikoo commented Nov 11, 2022

fabienbergerat commented Nov 11, 2022

Aljodomo commented Nov 14, 2022

fabienbergerat commented Dec 16, 2022

Willadaygo commented Apr 26, 2023

brunodeangelis commented Apr 26, 2023

zsavajji commented May 10, 2023

maximilliangeorge commented May 11, 2023

liulinboyi commented May 11, 2023 • edited Loading

davidwebca commented May 31, 2023

beejaz commented Jul 9, 2023

michaelpumo commented Jul 22, 2023

joseehilton147 commented Aug 1, 2023

JasonGotGithub commented Aug 28, 2023 • edited Loading

JasonGotGithub commented Sep 14, 2023

haoqunjiang commented Sep 27, 2023

maximilliangeorge commented Sep 27, 2023 • edited Loading

stoner-w commented Sep 27, 2023

daniluk4000 commented Sep 27, 2023

stoner-w commented Sep 27, 2023

daniluk4000 commented Sep 27, 2023

pikax commented Oct 20, 2023

fix: `<Suspense>` + `<Transition>` means mounted() runs too early #5952

fix: `<Suspense>` + `<Transition>` means mounted() runs too early #5952

Hugofgx9 commented Jul 7, 2022 •

edited

Loading

liulinboyi commented May 11, 2023 •

edited

Loading

JasonGotGithub commented Aug 28, 2023 •

edited

Loading

maximilliangeorge commented Sep 27, 2023 •

edited

Loading