-
Notifications
You must be signed in to change notification settings - Fork 90
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
Modal close button missing after 5.x #2529
Comments
@skjnldsv Any idea why that could happen? Found this when guiding @vinicius73 a bit through our frontend development stack. Seems to come from https://github.com/nextcloud/nextcloud-vue/blob/master/src/components/Actions/Actions.vue#L463 but I cannot really explain why this only happens in the modal, while the same combination of a single ActionButton works fine outside. Further this was introduced somewhere between 4.4.0 and 5.0.0. |
It is related to the last release of Vue Probably the slots are not yet available in the |
seems to be fixed with 2.6.14 which we are using 🤔 |
Bisecting showed #2385 as the commit causing the regression. Might be that the upstream change from functional to regular components now causes the rendering to happen earlier when the slot details are not fully available |
This seems to occur in viewer as well as on the documentation examples:
https://nextcloud-vue-components.netlify.app/#/Components/Modal
For some reason the single entry item for rendering the ActionButton seems to be missing:
l
The text was updated successfully, but these errors were encountered: