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

Display the custom login logo on the about modal #5685

Merged
merged 2 commits into from
Jun 13, 2019

Conversation

skateman
Copy link
Member

@skateman skateman commented Jun 7, 2019

This is an ugly hack that adds a class on each page's body element that will trigger the custom background image on the about modal. Eventually we would like to have a feature in PF for this.

@miq-bot add_label bug, graphics
@miq-bot add_reviewer @epwinchell
@miq-bot add_reviewer @himdel

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

@skateman
Copy link
Member Author

skateman commented Jun 7, 2019

@miq-bot add_label hammer/yes

Copy link
Contributor

@epwinchell epwinchell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested. Looks fine.

@skateman skateman force-pushed the about-modal-background branch from c60dc90 to 37884f7 Compare June 13, 2019 06:15
@miq-bot
Copy link
Member

miq-bot commented Jun 13, 2019

Checked commits skateman/manageiq-ui-classic@fe7a1e7~...37884f7 with ruby 2.3.3, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
0 files checked, 0 offenses detected
Everything looks fine. ⭐

@himdel himdel merged commit 1ce26ee into ManageIQ:master Jun 13, 2019
@himdel himdel self-assigned this Jun 13, 2019
@himdel himdel added this to the Sprint 114 Ending Jun 24, 2019 milestone Jun 13, 2019
simaishi pushed a commit that referenced this pull request Jun 13, 2019
Display the custom login logo on the about modal

(cherry picked from commit 1ce26ee)

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

Hammer backport details:

$ git log -1
commit 31b5c2648e15d66a414fe7e639a5eb6d540ac5a9
Author: Martin Hradil <[email protected]>
Date:   Thu Jun 13 10:08:19 2019 +0000

    Merge pull request #5685 from skateman/about-modal-background
    
    Display the custom login logo on the about modal
    
    (cherry picked from commit 1ce26eeb8d0c6d82fc4c3c02f4aacce6315f38bd)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1720242

@skateman skateman deleted the about-modal-background branch June 13, 2019 16:29
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