-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Fix missing axis labels when moving position #32628
Fix missing axis labels when moving position #32628
Conversation
💚 Build Succeeded |
@flash1293 We should probably backport all the way to 6.7 for inclusion in 6.7.1, though you’ll need to wait to merge that until the final 6.7 release is cut, since this isn’t a blocker. I’m not sure if this issue qualifies for backport to 6.6 though... I’d defer to @LeeDr on that |
Pinging @elastic/kibana-app |
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.
Tested that it fixes the issue on Chrome Linux for me. Can't say I completely understand the code, but it LGTM
Fixes #32059
The counter for the current position of the axis label during filtering didn't work correctly for vertical axes. I couldn't find another place which triggered this branch but the one outlined in the issue.
@lukeelmers Do you know how far we should backport this by any chance?