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 Bottlenecks timeline no data message #3458

Merged
merged 1 commit into from
Feb 22, 2018

Conversation

GregP
Copy link
Contributor

@GregP GregP commented Feb 22, 2018

Fixes Bottlenecks timeline data not available message display from warning to information.

Note: Originating BZ request states that an error message, in red, is being displayed. The message and background appear to be "reddish", but in fact are "yellow" per Patternfly use and definition. Message itself has been refactored to be an info type to alleviate any visual confusion.

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

Screen shot prior to fix:
bottlenecks no tm data warning msg prior to code fix

Screen shot post code fix, note Info message display:
bottlenecks no tm data info msg post code fix

@miq-bot
Copy link
Member

miq-bot commented Feb 22, 2018

Checked commit GregP@f8da544 with ruby 2.3.3, rubocop 0.52.0, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 🍰

@GregP
Copy link
Contributor Author

GregP commented Feb 22, 2018

@h-kataria Ready for review.

@h-kataria h-kataria added this to the Sprint 80 Ending Feb 26, 2018 milestone Feb 22, 2018
@h-kataria h-kataria merged commit 9e9ba41 into ManageIQ:master Feb 22, 2018
simaishi pushed a commit that referenced this pull request Mar 8, 2018
@simaishi
Copy link
Contributor

simaishi commented Mar 8, 2018

Gaprindashvili backport details:

$ git log -1
commit 108df0c5ef61cc43e78612c6d1fd8801510c67fb
Author: Harpreet Kataria <hkataria@redhat.com>
Date:   Thu Feb 22 17:33:53 2018 -0500

    Merge pull request #3458 from GregP/bottlenecks_tm_nodata_info_msg
    
    Fix Bottlenecks timeline no data message
    (cherry picked from commit 9e9ba4161748bab4c563801e541c3466b2cc3ee3)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1553288

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