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 Incorrect zoom out icon on C&U charts #13264

Merged
merged 1 commit into from
Dec 20, 2016

Conversation

PanSpagetka
Copy link
Contributor

@PanSpagetka PanSpagetka commented Dec 20, 2016

Incorrect zoom out icon on C&U charts.

Screenshots

Before:
zoom_out

After:
screencapture-localhost-3000-host-show-10000000000017-1482228658207

Links

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

Steps for Testing/QA

@miq-bot
Copy link
Member

miq-bot commented Dec 20, 2016

Checked commit PanSpagetka@bffbd73 with ruby 2.2.5, rubocop 0.37.2, and haml-lint 0.16.1
1 file checked, 1 offense detected

app/helpers/charting_helper.rb

@dclarizio dclarizio self-assigned this Dec 20, 2016
@dclarizio dclarizio merged commit 162811b into ManageIQ:master Dec 20, 2016
@dclarizio dclarizio added this to the Sprint 51 Ending Jan 2, 2017 milestone Dec 20, 2016
simaishi pushed a commit that referenced this pull request Jan 10, 2017
@simaishi
Copy link
Contributor

Euwe backport details:

$ git log -1
commit 047eeea3f8e2ba3dec745023f7cfc44b3a653613
Author: Dan Clarizio <[email protected]>
Date:   Tue Dec 20 07:11:03 2016 -0800

    Merge pull request #13264 from PanSpagetka/fix-zoom-out-icon
    
    Fix Incorrect zoom out icon on C&U charts
    (cherry picked from commit 162811b51dd4c21b9cbb2fa4cde925e222d349c8)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1411885

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