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

Height in p-chart is not applied if the responsive parameter is true #5748

Closed
fernandocode opened this issue May 16, 2018 · 2 comments
Closed
Assignees
Labels
Type: Bug Issue contains a bug related to a specific component. Something about the component is not working
Milestone

Comments

@fernandocode
Copy link

fernandocode commented May 16, 2018

I'm submitting a ... (check one with "x")

[x] bug report => Search github for a similar issue or PR before submitting
[ ] feature request => Please check if request is not on the roadmap already https://github.com/primefaces/primeng/wiki/Roadmap
[ ] support request => Please do not submit support request here, instead see http://forum.primefaces.org/viewforum.php?f=35

Plunkr Case (Bug Reports)
Please demonstrate your case at stackblitz by using the issue template below. Issues without a test case have much less possibility to be reviewd in detail and assisted.

https://stackblitz.com/edit/angular-zahjdv

Current behavior
If the responsive parameter is true (or default), the "height" property is not applied to the chart, and the default value 150 is applied. This also applies to "width".

Expected behavior
When you enter a value for height or width, it is expected to be applied to the component.
And in addition the only way to apply it, is putting "responsive" as false, which ends up causing the other property can not continue working responsively.

Minimal reproduction of the problem with instructions
Follow the comments in the example:
https://stackblitz.com/edit/angular-zahjdv

If they find this correction to be valid, I can send a PR.

@Danieleeee
Copy link

Danieleeee commented May 18, 2018

There are 5 releases that I report the problem of the height of the graphs and it is always ignored

@JacobSiegle
Copy link
Contributor

#5858

@cagataycivici cagataycivici self-assigned this Jun 6, 2018
@cagataycivici cagataycivici added the Type: Bug Issue contains a bug related to a specific component. Something about the component is not working label Jun 6, 2018
@cagataycivici cagataycivici added this to the 6.0.0-beta.1 milestone Jun 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Issue contains a bug related to a specific component. Something about the component is not working
Projects
None yet
Development

No branches or pull requests

4 participants