Inconsistent document descriptions in DynamicTexture class

Open Beginner friendly
#7,279 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
1/5
Estimated time
Under an hour
Newbie friendliness
75/100
Issue type
Documentation
Clarity
Clearly specified
Activity status
Quiet
Tech stack
javascript
Domain
documentation

Research direction

Open src/textures/DynamicTexture.js around lines 761-763 and review the document comment describing the missing drawFrame method. Update the description to match the DynamicTexture API that exists in Phaser 4, then verify the obsolete method reference is no longer present.

Written by the indexing model from the issue text.

Description

Version

  • Phaser Version: 4.0.0
  • Operating system:
  • Browser:

Description

https://github.com/phaserjs/phaser/blob/master/src/textures/DynamicTexture.js#L761-L763

     * You can pass in a string in which case it will look for a texture in the Texture
     * Manager matching that string, and draw the base frame. If you need to specify
     * exactly which frame to draw then use the method `drawFrame` instead.

But drawFrame method is not existed (removed already) in DynamicTexture class in phaser4.

Dominant language
JavaScript
Stars
40.3k
Forks
7.2k
PR merge metrics
No merged PRs in 30d

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from phaserjs/phaser

All issues in phaserjs/phaser

Similar issues

More JavaScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.