Mojtaba Samimi
Mojtaba Samimi
Please fetch `upstream/master` and merge in the branch. Then please update your https://jsfiddle.net/q2wt06d3/ using a build from `publish-dist/artifacts`. Thank you!
@alexturcea Are you interested in completing this PR? Could you try @eiriklv suggestion? https://github.com/plotly/plotly.js/pull/6452#discussion_r1630042980
> Pretty cool, doesn't seem like it was that hard to make this work > > We'll need to do something about the [default trace type](https://github.com/plotly/plotly.js/blob/3f338292575425242bdd6f0e0b3548c80d039028/src/plots/attributes.js#L9) - the two options...
I still think that a new attribute named `groupspacing` could be more flexible (and less confusing). See https://github.com/plotly/plotly.js/issues/5580#issuecomment-866166078.
Thanks for the PR. Seems like a good catch at first glance. However there are a number of baselines that change. So maybe one could consider enabling this option as...
@alexcjohnson I don't recall what `noCIdep` was about. `¯\_(ツ)_/¯` After the changes proposed in this PR, it looks like we need to adjust the `cartesian` test right above the line...
Reworked in #7475. Closing.
Closing. Please see new PR #7468 instead.
@martian111 Please fetch `upstream/master` and merge it into this branch. Thank you!
Thanks for the updates. This is looking great to me.