Skip to content

Conversation

@nikneym
Copy link
Contributor

@nikneym nikneym commented Jan 4, 2026

This is missing getContext method since I'm not sure how custom element methods are implemented in zigdom.

@nikneym nikneym added the zigdom label Jan 4, 2026
@karlseguin
Copy link
Collaborator

I don't understand how this is related to custom elements?

@nikneym
Copy link
Contributor Author

nikneym commented Jan 5, 2026

I don't understand how this is related to custom elements?/

Sorry for the misunderstanding, what I was trying to mean is to provide getContext function to HTMLCanvasElement.

@karlseguin
Copy link
Collaborator

There's a fn getContext() in https://github.com/lightpanda-io/browser/blob/zigdom/src/browser/webapi/element/html/Canvas.zig. You can remove the dummy RenderingContext2D from that file. Remove the registerTypes function (since you're explicitly adding RenderingContext2D to the bridge)..and that's it.

Base automatically changed from zigdom to main January 8, 2026 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants