Skip to content

Change from Chart.js to Plotly.js graph library#16

Merged
chrisdone-artificial merged 3 commits intomainfrom
harryroscoe/plotly-graph-lib
Jan 5, 2026
Merged

Change from Chart.js to Plotly.js graph library#16
chrisdone-artificial merged 3 commits intomainfrom
harryroscoe/plotly-graph-lib

Conversation

@harryroscoe-artificial
Copy link
Contributor

@harryroscoe-artificial harryroscoe-artificial commented Jan 2, 2026

Plotly has the zoom feature in both axes built-in alongside some other niceties and I was able to quite easily move the legend into the graph itself so it doesn't squash the graph into a tiny space when the legend gets big.

The legend itself is made slightly transparent so that the underlying graph can still be seen (also easier to pan and zoom with plotly to see what you want to despite the legend)

The config adapting from Chart.js to Plotly.js format was done by Claude but produces graphs that look right to me and nothing looks weird from my own review of it.

With some dummy data a playthrough of what this looks like:
https://github.com/user-attachments/assets/b4c1f1e4-77f2-4a89-bb9d-b60cb1ff047f

hpack = hp.hpack;
webook = pkgs.webhook;
hell = hell.packages.${system}.default;

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just added this because it is a language server for nix and my editor was complaining I didnt have one but can remove

@chrisdone-artificial
Copy link
Contributor

Cool, thanks!

@chrisdone-artificial chrisdone-artificial merged commit 8dd821a into main Jan 5, 2026
1 check passed
@chrisdone-artificial chrisdone-artificial deleted the harryroscoe/plotly-graph-lib branch January 5, 2026 09:20
@chrisdone-artificial
Copy link
Contributor

Deploying.

@chrisdone-artificial
Copy link
Contributor

Deployed. 🥳

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants