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

Swap the save/cancel buttons on the dialog editor to be consistent #3749

Merged
merged 1 commit into from
Apr 12, 2018

Conversation

skateman
Copy link
Member

@skateman skateman commented Apr 11, 2018

In every other form the Save button is the leftmost element on the bottom of the screen. However, the dialog editor somehow has it swapped, so changing it for the sake of consistency.

Before:
screenshot from 2018-04-11 13-13-23

After:
screenshot from 2018-04-11 13-12-57

@miq-bot add_reviewer @romanblanco
@miq-bot add_reviewer @himdel
@miq-bot add_label bug

@miq-bot miq-bot requested review from romanblanco and himdel April 11, 2018 11:14
@miq-bot miq-bot added the bug label Apr 11, 2018
@skateman
Copy link
Member Author

@miq-bot add_reviewer @epwinchell

@miq-bot miq-bot requested a review from epwinchell April 11, 2018 11:16
@miq-bot
Copy link
Member

miq-bot commented Apr 11, 2018

Checked commit skateman@0894e43 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
0 files checked, 0 offenses detected
Everything looks fine. 🍪

@epwinchell
Copy link
Contributor

@skateman looks fine

@miq-bot
Copy link
Member

miq-bot commented Apr 11, 2018

@epwinchell unrecognized command 'rm_reviewer', ignoring...

Accepted commands are: add_label, add_reviewer, assign, close_issue, move_issue, remove_label, rm_label, set_milestone

@skateman
Copy link
Member Author

@miq-bot assign @mzazrivec

@mzazrivec mzazrivec added this to the Sprint 84 Ending Apr 23, 2018 milestone Apr 12, 2018
@mzazrivec mzazrivec merged commit 8eef25b into ManageIQ:master Apr 12, 2018
@skateman skateman deleted the dialog-editor-reorder-buttons branch April 12, 2018 07:14
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.

5 participants