UI - Oversized Donut while exploring dev demo

[epicmaxco]

UI - Oversized Donut while exploring dev demo

[apertureless]

Sorry, my fault 🙈
With [email protected] I introduced dynamic styles for the div containing the canvas. However the 200% height was a typo.

Should be fixed with [email protected]

✌️

[smartapant]

@bdeanindy thank you for your investigation! We will check this thing after the weekend. Quite strange issue though

[bdeanindy]

So I reset my Git head back to when I first cloned, opened Statistics -> Charts and I see the same issue with all of the charts, they all have the wrapper to the <iframe> having a height: 200%.

So I am going to poke around a few more minutes to see if I can figure out where this is coming from (of course, I've never used Vue.js before, so this is a learning experience for me in several ways).

[bdeanindy]

Ok, after some archaeology, I discovered that this appears to be an issue with the Vue-ChartJS Base Charts having the height of all elements set to 200%. I am unsure if this is intentional, but I would think it is not.

The result of this is that all the charts used anywhere in your app are then doubled in height as compared to their parent element's height.

Would you like to close this issue and have me create an issue on their lib, keep this issue open and link to a new issue on that lib, or what precisely?

[bdeanindy]

@apertureless thanks for the quick fix.