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

scale service - respect new weight scale option for axes ordering #4094

Merged
merged 4 commits into from
Apr 4, 2017

Conversation

cizmiak
Copy link
Contributor

@cizmiak cizmiak commented Mar 31, 2017

#4092
Axes with undefined weight will have weight = 0 and will be drawn first, then will be drawn ordered axes by their weight.

@etimberg
Copy link
Member

I think it'd be a good idea to have a test for this so that we know that the weights are being correctly set into the layout system

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.

Per my comment about tests, marking this as requesting changes

Copy link
Member

@simonbrunel simonbrunel left a comment

Choose a reason for hiding this comment

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

+1 for unit tests

@etimberg etimberg merged commit 69dd0bd into chartjs:master Apr 4, 2017
@simonbrunel simonbrunel added this to the Version 2.6 milestone May 7, 2017
ericnkatz pushed a commit to ericnkatz/Chart.js that referenced this pull request May 24, 2017
…artjs#4094)

* respect new scale option 'order' when ordering scales

* scale service - respect new weight scale option for axes ordering

* added test for scale ordering by weight

* removed trailing spaces from layout weight scale order test
roicos pushed a commit to roicos/Chart.js that referenced this pull request Aug 21, 2017
…artjs#4094)

* respect new scale option 'order' when ordering scales

* scale service - respect new weight scale option for axes ordering

* added test for scale ordering by weight

* removed trailing spaces from layout weight scale order test
exwm pushed a commit to exwm/Chart.js that referenced this pull request Apr 30, 2021
…artjs#4094)

* respect new scale option 'order' when ordering scales

* scale service - respect new weight scale option for axes ordering

* added test for scale ordering by weight

* removed trailing spaces from layout weight scale order test
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.

3 participants