Skip to content

Comments

enh: Migrate to NcDialog#697

Merged
susnux merged 2 commits intomainfrom
enh/use-nc-dialog
Mar 22, 2024
Merged

enh: Migrate to NcDialog#697
susnux merged 2 commits intomainfrom
enh/use-nc-dialog

Conversation

@susnux
Copy link
Collaborator

@susnux susnux commented Jan 22, 2024

Use NcDialog instead of NcModal, so we can reduce the boilerplate code (styles etc).

Screenshots

sorry forgot to switch to English language

before after
image image

@susnux susnux added enhancement New feature or request 3. to review Waiting for reviews labels Jan 22, 2024
@susnux susnux changed the base branch from main to fix/nextcloud-vue January 22, 2024 22:47
@susnux susnux force-pushed the enh/use-nc-dialog branch from 89f256e to ae3cf13 Compare January 22, 2024 22:48
@susnux susnux force-pushed the fix/nextcloud-vue branch from 6c28edd to 18e283b Compare January 22, 2024 22:48
@susnux susnux force-pushed the enh/use-nc-dialog branch from ae3cf13 to a9f0c7c Compare January 22, 2024 22:48
Base automatically changed from fix/nextcloud-vue to main January 22, 2024 23:25
Copy link

@emoral435 emoral435 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🐘 Whether with a form or using our component, this works 👍

@susnux susnux force-pushed the enh/use-nc-dialog branch from a9f0c7c to 96bd433 Compare March 22, 2024 14:44
@susnux susnux requested a review from Pytal March 22, 2024 14:45
@susnux susnux force-pushed the enh/use-nc-dialog branch from 96bd433 to 70e96f2 Compare March 22, 2024 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants