Skip to content

Tilemaps: findByIndex, putTilesAt and swapByIndex document parameters they do not take - #7350

Open
darkdi wants to merge 1 commit into
phaserjs:masterfrom
darkdi:tilemap-doc-param-names
Open

Tilemaps: findByIndex, putTilesAt and swapByIndex document parameters they do not take#7350
darkdi wants to merge 1 commit into
phaserjs:masterfrom
darkdi:tilemap-doc-param-names

Conversation

@darkdi

@darkdi darkdi commented Aug 9, 2026

Copy link
Copy Markdown

Please do not update the README or Change Log, we will do this when we merge your PR.

This PR (delete as applicable)

  • Updates the Documentation

Describe the changes below:

Was reading the Tilemaps source next to the generated typings and phaser.d.ts hands TypeScript users swapByIndex(tileA, tileB, ...), while the method takes indexA and indexB and the sentence right above the tags already calls them that. Same drift in findByIndex, putTilesAt and the private setTilesets, in Tilemap, TilemapLayerBase and the FindByIndex component. I left types/ alone since it is generated, so tsgen should pick these up on the next run.

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.

1 participant