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

Fix preselect node in servers by roles tree under diagnostics #5382

Merged
merged 1 commit into from
Mar 27, 2019

Conversation

skateman
Copy link
Member

This is as inconsistent behavior, there should be a preselected node with its details shown on the right side. The right side data was all there, however, the appropriate node was not highlighted. Now when you navigate to the page, it should preselect the node.

Screenshot from 2019-03-27 10-58-12

@miq-bot add_reviewer @ZitaNemeckova
@miq-bot add_label bug, trees

@ZitaNemeckova
Copy link
Contributor

Before:
Screen Shot 2019-03-27 at 11 17 13 AM
After:
Screen Shot 2019-03-27 at 11 22 34 AM

LGTM 👍

@skateman Travis failure is related.

@skateman skateman force-pushed the server-roles-highlight branch from 6a893e4 to cc755a9 Compare March 27, 2019 11:26
This is as inconsistent behavior, there should be a preselected node
with its details shown on the right side. The right side data was all
there, however, the appropriate node was not highlighted.
@skateman skateman force-pushed the server-roles-highlight branch from cc755a9 to 5c0e8cf Compare March 27, 2019 12:42
@miq-bot
Copy link
Member

miq-bot commented Mar 27, 2019

Checked commit skateman@5c0e8cf with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
2 files checked, 0 offenses detected
Everything looks fine. 👍

@skateman
Copy link
Member Author

@miq-bot assign @mzazrivec

@mzazrivec mzazrivec added this to the Sprint 108 Ending Apr 1, 2019 milestone Mar 27, 2019
@mzazrivec mzazrivec merged commit 7ac053c into ManageIQ:master Mar 27, 2019
@skateman skateman deleted the server-roles-highlight branch March 27, 2019 15:17
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.

4 participants