Polish confirmation modal #10101
Assignees
Labels
Comments
|
I think we can replace our confirmation modal with https://nextcloud-vue-components.netlify.app/#/Components/NcDialog |
|
How does this revision look? |
it looks very good. I see a border on top of buttons, if that was added by you, you should remove it because as per design we dont have border above the buttons. Otherwise that looks great so please send a pull request and add me as reviewer |
|
ok perfect, thanks, yeah i added the border, will remove it and pull. |
keremsemiz
added a commit
that referenced
this issue
Mar 25, 2025
Styling fixes to confirmation modal.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Steps to reproduce
Expected behavior
The modal looks a bit unfinished.
For example there's a lot of padding to the top but no spacing to the action items.
Actual behavior
Mail app version
629a9fa
Mailserver or service
No response
Operating system
No response
PHP engine version
None
Web server
None
Database
None
Additional info
No response
The text was updated successfully, but these errors were encountered: