Description
#2936 fixes zero line logic for regular cartesian plots, but large sploms do their own thing for performance. As @etpinard points out, this should be udpated too:
No need this do in this PR (opening a new issue would be fine), but we'll need to propagate these changes to large-sploms' regl-based grid line logic:
plotly.js/src/traces/splom/base_plot.js
Lines 171 to 183 in b5f8b23
where I didn't just reuse the
axes.js
logic because I wanted to avoid looping over the traces (to compute thathasBarsOrFill
boolean). Thinking about this again, looping over traces isn't really a performance hit for sploms (I can't really imagine a user wanting to plot more 10 splom traces on the same graph).So it would be worth trying to expose and ♻️ this logic here for large-sploms.