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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

RouterView slot #26998

Closed
2 of 4 tasks
frasza opened this issue Apr 30, 2024 · 3 comments 路 Fixed by #27050
Closed
2 of 4 tasks

RouterView slot #26998

frasza opened this issue Apr 30, 2024 · 3 comments 路 Fixed by #27050

Comments

@frasza
Copy link
Contributor

frasza commented Apr 30, 2024

Describe the feature

hey 馃憢

recently I came across a special case where I had to pass a component to child NuxtPage and figured out that NuxtPage does not seem to support passing vslot.

Here is the documentation from Vue Router about that:
https://router.vuejs.org/guide/advanced/router-view-slot

Seems very useful functionality. Do you think it would be possible to support it with NuxtPage?

Cheers!

Additional information

  • Would you be willing to help implement this feature?
  • Could this feature be implemented as a module?

Final checks

@Mini-ghost
Copy link
Contributor

I've attempted to enable the default slot support in <NuxtPage>, and it appears to be functioning well so far. If the core team is open to adopting this feature, may I submit a PR to implement it?

@frasza
Copy link
Contributor Author

frasza commented May 2, 2024

I wonder if there are any specific usecases that would be against implementation of the feature. Hopefully we can land this.馃

@manniL
Copy link
Member

manniL commented May 3, 2024

We as a team think it is a great idea!

A PR is more than welcome 馃憤馃徎

The only thing to ensure is that it doesn't "break" Suspense and works with it 馃構

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Later
Development

Successfully merging a pull request may close this issue.

3 participants