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

Updates in lollipops #768

Merged
merged 12 commits into from
Apr 28, 2023
Merged

Updates in lollipops #768

merged 12 commits into from
Apr 28, 2023

Conversation

ASmirnov-HORIS
Copy link
Collaborator

@ASmirnov-HORIS ASmirnov-HORIS commented Apr 25, 2023

  • Update API for the dir parameter.
  • Expand scales to zero when lollipops are perpendicular to the x-axis.
  • Fix links in the future_changes.md.

@@ -168,12 +168,16 @@ object PlotUtil {
internal fun rangeWithExpand(
Copy link
Collaborator

Choose a reason for hiding this comment

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

Could you instead keep the size related stuff in lollipop geom widthSpan / heightSpan ?

@ASmirnov-HORIS ASmirnov-HORIS changed the title Fix scales for lollipops Updates in lollipops Apr 28, 2023
@alshan alshan merged commit a29e35e into master Apr 28, 2023
@ASmirnov-HORIS ASmirnov-HORIS deleted the lollipop branch May 2, 2023 10:12
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.

None yet

2 participants