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

Added possibility to limit the width of bars #32

Conversation

iAppDeveloper88
Copy link
Contributor

If a bar chart has a small amount of bars and scrolling is disabled the bars get very big. Currently the bars have to fill the total width of the chart.
See this example:
Screenshot BarChart

With my adjustments in this pull request the default behaviour stays the same. But you can configure to limit the width of the bars to get a result like this:
Screenshot BarChart1

Copy link
Owner

@francescoleoni98 francescoleoni98 left a comment

Choose a reason for hiding this comment

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

Thank you for your work!
I have just a question.
On line 30, the comment says that the width will not be larger than the width value, but if limitWidth is set to false, the width can be larger. Is it right?

@iAppDeveloper88
Copy link
Contributor Author

Yes, you are right. I should probably rewrite the comment later today

@francescoleoni98
Copy link
Owner

Thank you ;)

@francescoleoni98 francescoleoni98 merged commit fcd9dcf into francescoleoni98:main May 11, 2022
@iAppDeveloper88 iAppDeveloper88 deleted the feat-barchartconfiguration branch May 11, 2022 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants