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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Missing close icon for modals #2130

Closed
mejo- opened this issue Jan 24, 2022 · 5 comments
Closed

Missing close icon for modals #2130

mejo- opened this issue Jan 24, 2022 · 5 comments
Labels
1. to develop bug Something isn't working
Milestone

Comments

@mejo-
Copy link
Member

mejo- commented Jan 24, 2022

The close icon of all our modals is missing since we upgraded to vue-material-design-icons 5.0.0.

  • When opening a text document in a public folder share
  • When opening the help modal from within a text document

After some debugging it seems like the icon slot of ActionButton, as used by the @nextcloud/vuejs Modal component, is empty for some reason.

Downgrading vue-material-design-icons to 4.12.0 brings back the Close icon, but I honestly don't understand what's happening there.

@mejo- mejo- added bug Something isn't working bug: regression labels Jan 24, 2022
@mejo- mejo- added this to the Nextcloud 24 milestone Jan 24, 2022
@juliusknorr
Copy link
Member

We should check if this is still an issue now that @nextcloud/[email protected] is used where the upgrade of vie-material-design-icons was also included nextcloud-libraries/nextcloud-vue@44a4689

@mejo-
Copy link
Member Author

mejo- commented Mar 3, 2022

I just tested shared folders when reviewing something else - and indeed the bug is still there on latest Nextcloud 24 with latest Text 😢

@szaimen
Copy link
Contributor

szaimen commented Mar 3, 2022

I think the close button it is missing everywhere (whenever a modal is opened after the 5.0 upgrade)

@juliusknorr
Copy link
Member

@vinicius73 Looked into that at nextcloud-libraries/nextcloud-vue#2529

Possible fix is waiting for a review in nextcloud-libraries/nextcloud-vue#2532

@mejo-
Copy link
Member Author

mejo- commented Mar 24, 2022

This got fixed in @nextcloud/vue 5.1.1, which we're using since 9f8bfc9.

@mejo- mejo- closed this as completed Mar 24, 2022
Repository owner moved this from 🧭 Planning evaluation (don't pick) to ☑️ Done in 📝 Office team Mar 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop bug Something isn't working
Projects
Archived in project
Development

No branches or pull requests

3 participants