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

Honor 'display' parameter for button groups #1663

Merged

Conversation

mzazrivec
Copy link
Contributor

  1. In OPS UI, create a button group for Services, make sure it has the Display on Button property unchecked
  2. In Service UI, navigate to a service summary screen, see whether the display property is being honored.

Without this fix, the property would not get honored in Service UI.

After this fix, with display disabled:
Screenshot from 2020-08-26 16-56-46

With display enabled:
Screenshot from 2020-08-26 16-57-13

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

@miq-bot
Copy link
Member

miq-bot commented Aug 26, 2020

Checked commit mzazrivec@f9e99e1 with ruby 2.6.3, rubocop 0.69.0, haml-lint 0.28.0, and yamllint
0 files checked, 0 offenses detected
Everything looks fine. 🏆

@mzazrivec mzazrivec requested a review from himdel August 26, 2020 15:26
@himdel himdel self-assigned this Aug 26, 2020
@himdel himdel merged commit 9127abd into ManageIQ:master Aug 26, 2020
simaishi pushed a commit that referenced this pull request Aug 27, 2020
…isplay_group

Honor 'display' parameter for button groups

(cherry picked from commit 9127abd)
@simaishi
Copy link
Contributor

Jansa backport details:

$ git log -1
commit 65254786c0e5906861ee1cea7ee4f2de70889edb
Author: Martin Hradil <mhradil@redhat.com>
Date:   Wed Aug 26 15:38:43 2020 +0000

    Merge pull request #1663 from mzazrivec/honor_display_parameter_for_display_group

    Honor 'display' parameter for button groups

    (cherry picked from commit 9127abdaa3c423d354fc6a5d9209387e0e8d3ca2)

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

5 participants