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

[v0.5] Horizontal Bar Chart: only accept a single Y Axis #40

Closed
wants to merge 1 commit into from

Conversation

jentfoo
Copy link
Member

@jentfoo jentfoo commented Jan 23, 2025

Because of the horizontal layout two Y axis's do not make sense for this chart. Prior to this API change it would result in a confusing / bugged chart if a second Y-Axis is specified.

Because this is API breaking this wont be merged until we are ready to consider the v0.5.0 release.

@jentfoo jentfoo added bug Something isn't working enhancement New feature or request labels Jan 23, 2025
@jentfoo jentfoo self-assigned this Jan 23, 2025
@jentfoo jentfoo changed the title Horizontal Bar Chart: only accept a single Y Axis [v0.5] Horizontal Bar Chart: only accept a single Y Axis Jan 23, 2025
@jentfoo jentfoo force-pushed the jent/horizontal_bar_yaxis_opt branch from 00e2638 to b5fd66f Compare January 27, 2025 04:45
Because of the horizontal layout two Y axis's do not make sense for this chart. Prior to this API change it would result in a confusing / bugged chart if a second Y-Axis is specified.
@jentfoo jentfoo force-pushed the jent/horizontal_bar_yaxis_opt branch from b5fd66f to b38f188 Compare January 27, 2025 05:07
@jentfoo jentfoo marked this pull request as draft January 27, 2025 16:29
@jentfoo jentfoo mentioned this pull request Jan 27, 2025
@jentfoo
Copy link
Member Author

jentfoo commented Jan 27, 2025

Replaced with a single v0.5 PR: #44

@jentfoo jentfoo closed this Jan 27, 2025
@jentfoo jentfoo deleted the jent/horizontal_bar_yaxis_opt branch January 28, 2025 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant