Skip to content

Limit tags chart to leadercount#418

Merged
devmount merged 2 commits intodevmount:mainfrom
dillenger:main
Aug 21, 2023
Merged

Limit tags chart to leadercount#418
devmount merged 2 commits intodevmount:mainfrom
dillenger:main

Conversation

@dillenger
Copy link
Copy Markdown
Contributor

This limits the tags chart to the same number of entries which is used to limit the frequent contacts chart.
Prevents the chart from getting squeezed too much when you have more than 20 tags.

@devmount
Copy link
Copy Markdown
Owner

devmount commented Aug 8, 2023

Thank you for this contribution! 👏🏻 Thinking about this, I guess it would be even better to give this option a more general name (like maxBarCount) and apply it to all horizontal bar charts (since all horizontal bar charts can run into the problem of having too much entries on the y-axis). I will extend this PR and merge it once it's ready.

Again, thank you so much for this contribution!

@devmount devmount self-assigned this Aug 8, 2023
@devmount devmount added the enhancement Improve quality of existing features label Aug 8, 2023
@devmount devmount merged commit 8f18387 into devmount:main Aug 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Improve quality of existing features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants