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
lower z index of scale border so it draws behind the chart elements again #9412
lower z index of scale border so it draws behind the chart elements again #9412
Conversation
src/core/core.scale.js
Outdated
@@ -1599,8 +1599,8 @@ export default class Scale extends Element { | |||
_layers() { | |||
const me = this; | |||
const opts = me.options; | |||
const tz = opts.ticks && opts.ticks.z || 0; | |||
const gz = opts.grid && opts.grid.z || 0; | |||
const tz = opts.ticks && opts.ticks.z || -1; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should use valueOrDefault, because you can't define z as 0
now. Does the tick z need to be changed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The test wont fail if it isnt changed, but I thought since they where at the same z-index it would be logic to keep them at the same z-index
I think this needs a rebase for the CI to work. |
Could also use a regression test |
* Fix the title positions for center positioned axes * Handle center position for vertical axes * Tests
…apancies with scale borderColor
…enaleee/Chart.js into fix/axis-line-behind-bubble
It seems my knowledge on how to do a rebase is not good enough because it tries to merge existing commits into the master again, should I open a new clean pr to avoid issues or will this one be fine? |
@LeeLenaleee I do the following when rebasing: git checkout master
git pull upstream master
# only needed if your master bransh has some extra commits you need to get rid of
# git reset --hard upstream/master
git checkout <pr branch>
git rebase master
git push -f |
Closing in favour of #9426 |
Comming from #8936 resolves #9411
Since this was the behaviour in V2 and before v3.2 It should be considered a breaking change that the border of a scale draws above the chart elements by default, so it should be delayed to v4
https://codepen.io/leelenaleee/pen/ExmZXPa