Skip to content

Conversation

@ShGKme
Copy link
Contributor

@ShGKme ShGKme commented Nov 10, 2023

☑️ Resolves

🖼️ Screenshots

🏚️ Before 🏡 After
image image

@ShGKme ShGKme added bug Something isn't working 3. to review labels Nov 10, 2023
@ShGKme ShGKme self-assigned this Nov 10, 2023
@nickvergessen
Copy link
Member

The confirm on the delete should be red like in the web if that is easily doable?🤔

@ShGKme
Copy link
Contributor Author

ShGKme commented Nov 10, 2023

The confirm on the delete should be red like in the web if that is easily doable?🤔

I missed that a part of OC.dialogs was replaced with NcDialog in place in nextcloud/spreed#10812

This screenshot is not actual anymore. Only for the rest of OC.dialogs usage.

@ShGKme ShGKme merged commit 71a6327 into main Nov 10, 2023
@delete-merged-branch delete-merged-branch bot deleted the fix/oc-dialogs branch November 10, 2023 22:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add full-featured replacement or use original @nextcloud/dialogs

3 participants