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

Do not display GTL toolbars on non-GTL screens when navigating from the list #3617

Merged
merged 2 commits into from
Mar 15, 2018

Conversation

AparnaKarve
Copy link
Contributor

Currently, when navigating from the GO Definition list, the GTL buttons are displayed in the GO Definition Summary screen, which is not correct.
This issue is also visible on other summary screens like Custom Button, Custom Group(non-GTL screens) when the view is reloaded.
Navigating from the tree, however, displays the correct toolbar with no GTL buttons.

The PR fixes this inconsistency.

Before (List navigation shows GTL buttons in Summary screen) -
screen shot 2018-03-13 at 10 35 49 am

After (List navigation does not show GTL buttons in Summary screen) -
screen shot 2018-03-13 at 11 14 30 am

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

@AparnaKarve
Copy link
Contributor Author

@miq-bot add_label bug,generic objects

@miq-bot
Copy link
Member

miq-bot commented Mar 13, 2018

Checked commits AparnaKarve/manageiq-ui-classic@f2351bc~...7957c0b 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. 👍

@mzazrivec mzazrivec self-assigned this Mar 15, 2018
@mzazrivec mzazrivec added this to the Sprint 82 Ending Mar 26, 2018 milestone Mar 15, 2018
@mzazrivec mzazrivec merged commit ff32cf8 into ManageIQ:master Mar 15, 2018
@AparnaKarve AparnaKarve deleted the bz1545835_make_tb_consistent branch March 15, 2018 16:36
simaishi pushed a commit that referenced this pull request Mar 16, 2018
Do not display GTL toolbars on non-GTL screens when navigating from the list
(cherry picked from commit ff32cf8)

https://bugzilla.redhat.com/show_bug.cgi?id=1557384
@simaishi
Copy link
Contributor

Gaprindashvili backport details:

$ git log -1
commit 89a0a0d8ee6c1c9413d927ad17192a67bec675a4
Author: Milan Zázrivec <[email protected]>
Date:   Thu Mar 15 10:08:20 2018 +0100

    Merge pull request #3617 from AparnaKarve/bz1545835_make_tb_consistent
    
    Do not display GTL toolbars on non-GTL screens when navigating from the list
    (cherry picked from commit ff32cf87b8502daed169a1ede79478af317d3dfc)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1557384

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