Skip to content

docs(sprite): SpriteAnimationClip name, fps, loop property JSDoc#8680

Merged
willeastcott merged 1 commit into
mainfrom
docs/sprite-animation-clip-property-jsdoc
May 4, 2026
Merged

docs(sprite): SpriteAnimationClip name, fps, loop property JSDoc#8680
willeastcott merged 1 commit into
mainfrom
docs/sprite-animation-clip-property-jsdoc

Conversation

@willeastcott

Copy link
Copy Markdown
Contributor

Summary

Adds brief JSDoc summaries for the public name, fps, and loop fields on SpriteAnimationClip, alongside existing @type tags. These properties are user-visible (e.g. via SpriteComponent#addClip and playback) and surfaced in the API reference after the class-fields refactor; they should be documented as first-class API rather than @ignore or @private.

Changes

  • SpriteAnimationClip: one-line description per field, aligned with constructor / addClip parameter wording; notes that negative fps plays backwards.

Public API

Documentation only; runtime behavior unchanged.

Co-authored-by: Cursor <cursoragent@cursor.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR improves the public API documentation for SpriteAnimationClip by adding short JSDoc summaries to existing public fields. It fits the codebase’s API-doc conventions by making user-visible sprite animation properties read clearly in generated reference docs without changing runtime behavior.

Changes:

  • Added a short summary for the public name field.
  • Added a short summary for the public fps field, including that negative values play backwards.
  • Added a short summary for the public loop field.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@willeastcott willeastcott merged commit c0428ee into main May 4, 2026
12 checks passed
@willeastcott willeastcott deleted the docs/sprite-animation-clip-property-jsdoc branch May 4, 2026 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation related enhancement Request for a new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants