Bug 1393011 - New secondary charts don't match the prior version
Summary: New secondary charts don't match the prior version
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.7.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: GA
: 5.8.0
Assignee: Robin Knaur
QA Contact: Satyajit Bulage
URL:
Whiteboard:
Depends On:
Blocks: 1393980
TreeView+ depends on / blocked
 
Reported: 2016-11-08 16:58 UTC by Dan Clarizio
Modified: 2017-06-12 16:30 UTC (History)
7 users (show)

Fixed In Version: 5.8.0.0
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1393980 (view as bug list)
Environment:
Last Closed: 2017-06-12 16:30:07 UTC
Category: ---
Cloudforms Team: CFME Core
Target Upstream Version:


Attachments (Terms of Use)
5.7 charts with large secondary chart and truncated legends (34.55 KB, image/png)
2016-11-08 16:58 UTC, Dan Clarizio
no flags Details
5.6 charts with small secondary chart and readable legends (15.73 KB, image/png)
2016-11-08 17:06 UTC, Dan Clarizio
no flags Details

Description Dan Clarizio 2016-11-08 16:58:26 UTC
Created attachment 1218616 [details]
5.7 charts with large secondary chart and truncated legends

Description of problem:
The charts displayed for Clusters/Deployment Roles and for Hosts/Nodes have a secondary chart below them showing Avg Running Host/Avg Running VMs.  These charts are too big and the legend text strings are truncated.

Version-Release number of selected component (if applicable):
5.7.0

How reproducible:
View the charts

Steps to Reproduce:
1. Go to a Cluster or a Host that has C&U data
2. Display the Utilization charts

Actual results:
See screenshot of 5.7 charts

Expected results:
See screenshot of 5.6 charts

Comment 2 Dan Clarizio 2016-11-08 17:06:26 UTC
Created attachment 1218618 [details]
5.6 charts with small secondary chart and readable legends

Comment 4 Robin Knaur 2016-11-09 09:25:11 UTC
https://github.com/ManageIQ/manageiq/pull/12521

Comment 5 CFME Bot 2016-11-09 18:36:35 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/1e92365c11c2ba98e9b33320cb36c111e62ac3f6

commit 1e92365c11c2ba98e9b33320cb36c111e62ac3f6
Author:     PanSpagetka <rknaur>
AuthorDate: Wed Nov 9 10:16:31 2016 +0100
Commit:     PanSpagetka <rknaur>
CommitDate: Wed Nov 9 10:16:31 2016 +0100

    Fix CU secondary chart labels and size
    https://bugzilla.redhat.com/show_bug.cgi?id=1393011

 app/assets/javascripts/miq_application.js | 4 ++--
 lib/report_formatter/c3.rb                | 5 +++--
 2 files changed, 5 insertions(+), 4 deletions(-)

Comment 6 CFME Bot 2016-11-09 23:21:20 UTC
New commit detected on ManageIQ/manageiq/euwe:
https://github.com/ManageIQ/manageiq/commit/6c695e7db9b20c058dfaa32d77a462ff68727c4b

commit 6c695e7db9b20c058dfaa32d77a462ff68727c4b
Author:     Dan Clarizio <dclarizi>
AuthorDate: Wed Nov 9 10:32:50 2016 -0800
Commit:     Oleg Barenboim <chessbyte>
CommitDate: Wed Nov 9 18:20:22 2016 -0500

    Merge pull request #12521 from PanSpagetka/fix-cu-flat-chart
    
    Fix CU secondary chart labels and size
    (cherry picked from commit 8d3813f942a39f48f48c21ba3670b5d688e524cb)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1393011

 app/assets/javascripts/miq_application.js | 4 ++--
 lib/report_formatter/c3.rb                | 5 +++--
 2 files changed, 5 insertions(+), 4 deletions(-)

Comment 9 Satyajit Bulage 2017-04-18 06:17:41 UTC
Utilization Charts are displaying without truncating the text for Cluster or Host. 

Verified Version: 5.8.0.10-beta1.20170411212748_e47d319


Note You need to log in before you can comment on or make changes to this bug.