[PATCH] Improvement for logarithmic scatter axis tick labels, avoid overlapping text

[jensb]

trafficstars

Hello,

I created a patch for Plotly which impvoes display of axis tick labels for logarithmic axes of scatter plots. More preciesly, it avoids overlapping labels much better (IMHO) than before.

The issue and the patch is described in more detail here: https://stackoverflow.com/questions/70926979/skip-overlapping-tick-labels-in-plotly-javascript/70953790#70953790

Do you think this has a chance to be merged? Then I'll create a PR for the master branch. If this needs more work, please advise.

Thank you!

[archmoj]

That looks interesting. Looking forward to reviewing your PR.

[jensb]

Sure! However, I need to ask for some assistance.

I set up the devel environment and I can run npm start to boot up. But on the first Tabs.reload() I get a syntax error in plotly.js. This is the offending line:

This is how the line looks after the first Tabs.reload():

There seems to be a charset issue in the reload feature. What am I doing going wrong?

I'm using Chrome 98.0.4758.80 on Linux, as well as NPM 8.3.1, Node 16.14.0.

[jensb]

I managed to patch Plotly anyway: https://github.com/plotly/plotly.js/pull/6120

[jensb]

Any comments, progress, review? Anything I need to do to help merging this? Yes, there are some failing tests, but this is obviously because I am (slightly) changing Plotly's behaviour, intentionally ... for the better, I hope.