Skip to content

Update RoundedBarChartRenderer.kt #363

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Paget96
Copy link

@Paget96 Paget96 commented Apr 24, 2025

Changelog - RoundedBarChartRenderer Improvements

Added: Auto Full Radius Mode
Introduced a new setUseAutoFullRadius(true) function that enables automatic calculation of full corner radius for bars.

When enabled, the radius dynamically adapts based on the bar’s current screen width and height, ensuring fully rounded corners regardless of zoom level or chart scale.

This feature improves visual consistency, especially when zooming in/out, as the rounding remains proportional to each bar’s dimensions.

Fixed: Marker Not Showing on Highlight
Previously, markers were not appearing when bars were highlighted due to missing positional metadata.

This has been resolved by ensuring setHighlightDrawPos(...) is called inside drawHighlighted(), which correctly updates the Highlight object with the marker's draw position.

Now, the custom and other markers will be properly displayed above highlighted bars, even when using this renderer.

Changelog - RoundedBarChartRenderer Improvements

Added: Auto Full Radius Mode
Introduced a new setUseAutoFullRadius(true) function that enables automatic calculation of full corner radius for bars.

When enabled, the radius dynamically adapts based on the bar’s current screen width and height, ensuring fully rounded corners regardless of zoom level or chart scale.

This feature improves visual consistency, especially when zooming in/out, as the rounding remains proportional to each bar’s dimensions.

Fixed: Marker Not Showing on Highlight
Previously, markers were not appearing when bars were highlighted due to missing positional metadata.

This has been resolved by ensuring setHighlightDrawPos(...) is called inside drawHighlighted(), which correctly updates the Highlight object with the marker's draw position.

Now, the custom and other markers will be properly displayed above highlighted bars, even when using this renderer.
@hannesa2
Copy link
Collaborator

There is an issue with pull requests from a fork, which prevent a properly running CI. The reason is a missing token on fork side to be able to push on screenshot compare job.
At least #365 makes it work for successful runs

@Paget96 Please rebase on recent master, to be able to run the CI at least on successful jobs

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.

2 participants