Skip to content

Conversation

@kurkle
Copy link
Member

@kurkle kurkle commented Jan 2, 2020

Fixes: #6811

etimberg
etimberg previously approved these changes Jan 2, 2020
Copy link
Member

@etimberg etimberg left a comment

Choose a reason for hiding this comment

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

Should this have a test too?

@kurkle
Copy link
Member Author

kurkle commented Jan 2, 2020

Added a test, used valueOrDefault and removed redundant !isNaN checks.

@etimberg etimberg merged commit 03f65d4 into chartjs:master Jan 3, 2020
@kurkle kurkle deleted the linear-suggested branch February 19, 2020 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Y-axis without data ignores suggestedMin/Max (regression 2.9)

4 participants