Skip to content

Conversation

kurkle
Copy link
Member

@kurkle kurkle commented May 4, 2019

Allow changing of drawing order by z-index. Option available for gridLines and ticks initially.

Demo

Alternative to: #6226

Closes: #2808
Closes: #3724
Closes: #3738
Closes: #3722
Closes: #3957

@simonbrunel simonbrunel changed the title Layers (z-index) Implement layers (z-index) for layout items May 4, 2019
@benmccann
Copy link
Contributor

Cool idea! I'd probably call it zIndex rather than just z, but curious what other think about the naming

@simonbrunel
Copy link
Member

I prefer z since I don't think of any other use for that option name and usually prefer shorter names (I'm also familiar with z in QML).

@simonbrunel simonbrunel requested review from nagix and etimberg May 6, 2019 20:46
etimberg
etimberg previously approved these changes May 7, 2019
Copy link
Contributor

@nagix nagix left a comment

Choose a reason for hiding this comment

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

Overall, Looks good.

benmccann
benmccann previously approved these changes May 8, 2019
@kurkle
Copy link
Member Author

kurkle commented May 8, 2019

Cool idea!

You have to thank @simonbrunel for the idea 👍

@simonbrunel simonbrunel added this to the Version 2.9 milestone May 9, 2019
@simonbrunel simonbrunel merged commit 2a96d83 into chartjs:master May 9, 2019
@kurkle kurkle mentioned this pull request May 9, 2019
@SimonCraenen
Copy link

When will this be released?

@jbee
Copy link

jbee commented Sep 20, 2019

Waiting for the feature as well.

exwm pushed a commit to exwm/Chart.js that referenced this pull request Apr 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
7 participants