Skip to content

Conversation

@MUSKANNISHAD
Copy link
Contributor

@MUSKANNISHAD MUSKANNISHAD commented Dec 26, 2025

This PR adds an example demonstrating how to create and copy Path2D paths.

Re-opening after an accidental branch reset while rebasing onto the latest main.
The content change itself is unchanged.

Happy to iterate based on feedback.

Fixes #41793

@MUSKANNISHAD MUSKANNISHAD requested a review from a team as a code owner December 26, 2025 04:28
@MUSKANNISHAD MUSKANNISHAD requested review from sideshowbarker and removed request for a team December 26, 2025 04:28
@github-actions github-actions bot added Content:WebAPI Web API docs size/s [PR only] 6-50 LoC changed labels Dec 26, 2025
Comment on lines +48 to +50
First, `path1` creates a rectangle.
Then, `path2` is created as a copy of `path1`.
After that, `path2` is extended with a circle.
Copy link
Contributor

Choose a reason for hiding this comment

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

[mdn-linter] reported by reviewdog 🐶

Suggested change
First, `path1` creates a rectangle.
Then, `path2` is created as a copy of `path1`.
After that, `path2` is extended with a circle.
First, `path1` creates a rectangle.
Then, `path2` is created as a copy of `path1`.
After that, `path2` is extended with a circle.


ctx.stroke(path2);
```
## Specifications
Copy link
Contributor

Choose a reason for hiding this comment

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

[mdn-linter] reported by reviewdog 🐶

Suggested change
## Specifications
## Specifications

## See also

- {{domxref("CanvasRenderingContext2D")}}

Copy link
Contributor

Choose a reason for hiding this comment

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

[mdn-linter] reported by reviewdog 🐶

Suggested change

@MUSKANNISHAD
Copy link
Contributor Author

Hi maintainers,

I’ve verified locally that there are no markdown, front-matter, or Prettier lint issues.
The remaining CI failure appears to be due to pending workflow approvals for pull_request_target, which I understand requires maintainer approval.

Please let me know if any changes are needed from my side. Thanks!

@sideshowbarker sideshowbarker removed their request for review December 26, 2025 04:37
@Josh-Cena Josh-Cena self-requested a review January 8, 2026 06:20
@github-actions
Copy link
Contributor

github-actions bot commented Jan 9, 2026

Preview URLs

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

Labels

Content:WebAPI Web API docs size/s [PR only] 6-50 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Path2: Example is confusing (or not adequately explained)

1 participant