You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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.
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.
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.
gvwilson
changed the title
[PATCH] Improvement for logarithmic scatter axis tick labels, avoid overlapping text
improve logarithmic scatter axis tick labels, avoid overlapping text
Aug 9, 2024
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!
The text was updated successfully, but these errors were encountered: