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

V8: Fix button style when opening doctype in infinite editing with unsaved changes #7003

Conversation

kjac
Copy link
Contributor

@kjac kjac commented Oct 31, 2019

Prerequisites

  • I have added steps to test this contribution in the description below

Description

The button style should probably be warning for the dialog that alerts you how changes will be discarded if you open the doctype in infinite editing without saving your changes first:

image

So... this PR updates the button style:

image

@poornimanayar
Copy link
Contributor

Thanks for this @kjac!

Poornima

Copy link
Contributor

@emmaburstow emmaburstow left a comment

Choose a reason for hiding this comment

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

This works! The only thing I will say is that the message is a little unclear so if you fancied doing anything about that, be my guest ;)

@emmaburstow emmaburstow changed the base branch from v8/dev to v8/contrib January 16, 2020 14:19
@emmaburstow emmaburstow added community/pr release/8.7.0 release/no-notes This is too small to add to the release notes or fixed after a beta/RC type/feature hacktoberfest/contrib-2019 labels Jan 30, 2020
@emmaburstow emmaburstow merged commit a41aa26 into umbraco:v8/contrib Jan 30, 2020
@kjac kjac deleted the v8/fix/infinite-edit-doctype-warning-style-dialog-button branch January 30, 2020 14:18
@nul800sebastiaan nul800sebastiaan removed the release/no-notes This is too small to add to the release notes or fixed after a beta/RC label Mar 2, 2020
@nul800sebastiaan nul800sebastiaan added the category/ui User interface label Jun 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants