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

Bug 1460122 - Only use confirm-on-exit on some forms #1674

Merged
merged 1 commit into from
Jun 9, 2017

Conversation

spadgett
Copy link
Member

@spadgett spadgett commented Jun 9, 2017

Use confirm-on-exit only on forms where the left navigation is visible or where you might have typed a lot of content, such as Edit YAML.

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1460122

@spadgett spadgett requested a review from jwforres June 9, 2017 15:51
@spadgett spadgett changed the title [WIP] Bug 1460142 - Only use confirm-on-exit on some forms Bug 1460142 - Only use confirm-on-exit on some forms Jun 9, 2017
@spadgett
Copy link
Member Author

spadgett commented Jun 9, 2017

This changes only the in-place environment variable editors and the edit YAML action to use confirm-on-exit.

@jwforres PTAL

Use confirm-on-exit only on forms where the left navigation is visible or
where you might have typed a lot of content, such as Edit YAML.
@spadgett spadgett changed the title Bug 1460142 - Only use confirm-on-exit on some forms Bug 1460122 - Only use confirm-on-exit on some forms Jun 9, 2017
@jwforres
Copy link
Member

jwforres commented Jun 9, 2017

[merge]

@openshift-bot
Copy link

Evaluated for origin web console merge up to 0ae3bbf

@openshift-bot
Copy link

openshift-bot commented Jun 9, 2017

Origin Web Console Merge Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin_web_console/1492/) (Base Commit: 513fa44)

@openshift-bot openshift-bot merged commit 9e2599c into openshift:master Jun 9, 2017
@spadgett spadgett deleted the rollback-confirm-on-exit branch June 10, 2017 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants