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

Fixes #29734 - Fix bar chart on config report page #7645

Merged
merged 1 commit into from May 13, 2020

Conversation

xprazak2
Copy link
Contributor

No description provided.

@theforeman-bot
Copy link
Member

Issues: #29734

@tbrisker
Copy link
Member

Much better, but looks like the chart is still overflowing a bit:
image
Also pushed a rebase on top of latest develop for you

@ekohl
Copy link
Member

ekohl commented May 12, 2020

I first thought it was a duplicate of https://projects.theforeman.org/issues/26090, but looks like it's just a similar issue on a different page.

@xprazak2
Copy link
Contributor Author

Thanks, donut chart should be now contained within the card as well.

@tbrisker tbrisker self-assigned this May 13, 2020
Copy link
Member

@tbrisker tbrisker left a comment

Choose a reason for hiding this comment

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

Thanks @xprazak2 !

@tbrisker tbrisker merged commit f79a909 into theforeman:develop May 13, 2020
<ChartBox
{...chartBoxProps}
type="bar"
chart={{ data: statusChartData, id: 'report-status' }}
title={__('Report Status')}
/>
</Col>
<Col md={4}>
<Col md={2}>
Copy link
Member

Choose a reason for hiding this comment

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

This makes it very hard to read on a smaller screen:
Screen Shot 2020-06-01 at 15 53 36

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants