Skip to content

wip: replace canvas with @napi-rs/canvas#148

Open
elliots wants to merge 7 commits intoSeanSobey:masterfrom
elliots:feat/othercanvas
Open

wip: replace canvas with @napi-rs/canvas#148
elliots wants to merge 7 commits intoSeanSobey:masterfrom
elliots:feat/othercanvas

Conversation

@elliots
Copy link

@elliots elliots commented Aug 20, 2024

just in case anyone else is interested.

no system dependencies, and prebuilt for arm (m1)

barely tested... but the example renders

image

@capaj
Copy link

capaj commented Dec 8, 2024

Just tried it in Bun and it works gr8, looks good to me on my chart here:
chart

@StoneCypher
Copy link

Interested

@Bhager01
Copy link

@elliots I'm definitely interested in this! Have you continued to use it? Curious how it is working for you.

@SeanSobey
Copy link
Owner

SeanSobey commented Feb 27, 2025

@elliots Thank you for your contribution! I have picked up working on this project again and am going to release a version soon working on canvas@3.1.0.

The idea of a much more isolated canvas implementation is very appealing, there have been many bugs and issues with platform specific 'considerations', including node-gyp. That said that project is still pre-release, and also does not support some features we need (like animations, recently). I will keep my eye on it closely, maybe have a look at a way to make the canvas implementation an option!

@Bhager01
Copy link

That's great to hear @SeanSobey ! I've been using this package for years now and I'm glad to see you'll be updating it to work with Canvas 3.1.0. Will you also be updating it to work with the newest chart.js (v4) or is that too big of an undertaking? Thanks again for your work on this!

@SeanSobey
Copy link
Owner

@Bhager01 Thanks! Indeed that's the plan, I already have a WIP version running on chart.js 4.x, there are quite a lot of integration changes required but its looking good for a release soon.

@Bhager01
Copy link

@Bhager01 Thanks! Indeed that's the plan, I already have a WIP version running on chart.js 4.x, there are quite a lot of integration changes required but its looking good for a release soon.

@SeanSobey I just saw you pushed out a new version! Thank you so much! I look forward to trying it out!

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.

6 participants