Skip to content

docs: Fix typos#3764

Merged
spydon merged 1 commit intoflame-engine:mainfrom
bensengupta:patch-1
Nov 9, 2025
Merged

docs: Fix typos#3764
spydon merged 1 commit intoflame-engine:mainfrom
bensengupta:patch-1

Conversation

@bensengupta
Copy link
Copy Markdown
Contributor

@bensengupta bensengupta commented Nov 9, 2025

Description

Fixes some small typos in the docs

Checklist

  • I have followed the Contributor Guide when preparing my PR.
  • I have updated/added tests for ALL new/updated/fixed functionality.
  • I have updated/added relevant documentation in docs and added dartdoc comments with ///.
  • I have updated/added relevant examples in examples or docs.

Breaking Change?

  • Yes, this PR is a breaking change.
  • No, this PR is not a breaking change.

Related Issues

Corrected the y-coordinate in the Vector2 example and updated the factory method name for creating RectangleComponent.
Copy link
Copy Markdown
Member

@spydon spydon left a comment

Choose a reason for hiding this comment

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

None of these changes are correct.

Comment thread doc/flame/components.md
Comment thread doc/flame/components.md
@bensengupta
Copy link
Copy Markdown
Contributor Author

Hi @spydon, I've left comments explaining my change. Are you able to take another look, or expand on what is incorrect?

@bensengupta bensengupta marked this pull request as ready for review November 9, 2025 17:44
@spydon
Copy link
Copy Markdown
Member

spydon commented Nov 9, 2025

Hi @spydon, I've left comments explaining my change. Are you able to take another look, or expand on what is incorrect?

Ah, true, they are actually correct! Sloppy review by me.

@spydon spydon changed the title docs: fix typos docs: Fix typos Nov 9, 2025
@spydon spydon merged commit 41e29a7 into flame-engine:main Nov 9, 2025
9 of 12 checks passed
nickf2k pushed a commit to nickf2k/flame that referenced this pull request Nov 16, 2025
Fixes some small typos in the docs
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