microsoft / microsoft/arcade-grid

Wall support for grid.moveWithButtons()

Open
#3 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
TypeScript
Stars
3
Forks
4
Avg merge
21d 21h
Merged PRs (30d)
1

Description

It looks like the grid.moveWithButtons() does not support walls on the tilemap. Is it possible to get it to work with walls?

Tech bits that I tried:

I wanted to see if I can solve this on my own, however it looks like the only way to check walls is to ask TileMapData: https://github.com/microsoft/pxt-common-packages/blob/master/libs/game/tilemap.ts#L196

And that you cannot get TileMapData directly from the TileMap because it's protected:
https://github.com/microsoft/pxt-common-packages/blob/master/libs/game/tilemap.ts#L232

Any suggestion to get pass this issue?

Contributor guide

No contributing guide indexed for this repository

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.

Research direction

Start at grid.moveWithButtons() and compare its tilemap access with the TileMapData and TileMap definitions in libs/game/tilemap.ts, especially the referenced lines. Confirm how wall data is exposed and define done as movement respecting tilemap walls; the issue names no test or specific implementation file.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
game-dev
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.