Mojtaba Samimi
Mojtaba Samimi
@mojoaxel we are planning next plotly.js minor (possibly next week) to include new features like this one. Do you have time to address the remaining comments, so that we could...
Interesting PR for shapes and selections. But I started wondering perhaps we should add a `categoryshift` option to Cartesian axes instead?
I am wondering if adding a `categoryselectionstart`, `categoryselectionend`, `categoryshapestart` and `categoryshapeend` to cartesian axes is a better option?
Let's ask @alexcjohnson to review :mag_right:
Considering https://github.com/plotly/plotly.js/pull/7010#issuecomment-2147726890 discussion, two attributes are needed for start and the end i.e. instead of `xshift` which is already implemented we need `x0shift` and `x1shift`.
#6347 seems related to this PR. No?
> > #6347 seems related to this PR. No? > > Yes. The current PR may be part of the groundwork required for #6347. Do you think any additional changes...
Thanks for the feedback @stephprobst. It's good to know that -0.5 and 0.5 values are sufficient in your use cases. With that I think rounding start and end values gives...
@my-tien There are not conflicts with master; but please fetch & merge `upstream/master` into this branch.
@stephprobst Do you have a use case for adding these options to selections? If no, I'd suggest not to implement it in this PR.