-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Setting height on chart creates a larger style tag surrounding it #454
Comments
There is some additional padding of around 14px to give some breathing space to the chart. |
A new property |
Thanks that would help a lot. It's hard to design panels around charts if you can't control their size. Do you really need this at all though? Seems like people can easily add their own wrapper element and css styles if they want (eg: for padding) |
Well, I made a mistake in the beginning by not providing inner padding which would give some breathing space by default. Now, changing that seems difficult. So, I provided this new option to remove the extra height. |
😊 I understand, I mean you might not need the style tag at all since anybody can just wrap the chart into their own div tag and place a margin |
Hey @mrabaev48 what exactly is your issue? The picture does not explain it... |
Image
Explanation
What is the behavior you expect?
Setting height on the chart (380px in this case) I expect the resulting chart to be exactly that height
What is happening instead?
A style tag is created surrounding the chart and a
min-height
style is set to larger than the height (394px in this case).The text was updated successfully, but these errors were encountered: