Skip to content

Conversation

codyebberson
Copy link

@codyebberson codyebberson commented Aug 15, 2023

@h-joo h-joo self-requested a review August 16, 2023 15:16
@codyebberson
Copy link
Author

Hi @h-joo - anything I can do to improve this?

@codyebberson
Copy link
Author

codyebberson commented Sep 29, 2023

@niloc132 - thank you for the review

I added the function definition to the CanvasPathMethods as well. Good catch.

This now follows the exact same pattern as rect in CanvasPathMethods, BaseRenderingContext2D, and Path2D.

@codyebberson
Copy link
Author

Hi @h-joo and @niloc132 - Anything I can do to move this along?

@niloc132
Copy link
Contributor

Sorry, I'm just a contributor, I don't have any special powers to get patches merged.

@zbynek
Copy link
Contributor

zbynek commented May 6, 2025

@codyebberson I think this can be closed since 638e7c8 added these methods

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.

3 participants