craft

craft copied to clipboard

Wrong sort order for dropped mini blocks

3

 The sandstone mini block should be obscured by the block to the south of the Agent, or it should spawn more centered on the grid square.

joshlory

Closed door and iron door assets should have ambient occlusion shadow at base

4

We can't apply the AO shadow to the base of the door because it needs to disappear at the start of the door open animation and at the end of...

joshlory

 Repros on https://studio.code.org/s/minecraft/stage/1/puzzle/9. Sample code: 

joshlory

After https://github.com/code-dot-org/craft/issues/459 is addressed, we can replace all static methods like `Position.add(left, right)` with instance methods like `position.add(other)`

Hamms

For backwards compatibility during the transition from `[x, y]` to `new Position(x, y)`. As of https://github.com/code-dot-org/craft/pull/453, we should be able to exclusively use `.x` and `.y` rather than `[0]` and...

Hamms

The visual effect isn't too obvious, but this may represent a bigger perf issue. Are we re-generating all redstone wire sprites on every action plane change?

joshlory

Opacity of blocks does not return to normal after the player moves from beyond a block

1

Found in: https://levelbuilder-studio.code.org/s/craft17/stage/1/puzzle/12 Not a huge issue at this point, but the blocks become see through when a player is standing on the back side of blocks. When the player...

v-jaarba

Found In: https://levelbuilder-studio.code.org/s/craft17/stage/1/puzzle/12 When Player creates an infinite loop inside level 12 and then switches level and returns back, the Finish button is not displayed. Repro Steps: 1. Start level...

v-samtha

Trap these messages so they don't end up in TravisCI output: ``` Phaser.Cache.getImage: Key "blocks" not found in Cache. Cannot set frameName: Grass Phaser.Cache.getImage: Key "blocks" not found in Cache....

joshlory

Line coverage can vary slightly even with no code changes

3

PR https://github.com/code-dot-org/craft/pull/52 attempted to reduce variance in code coverage metrics due to random animation timing. This helped, but we're still seeing tiny variation in line coverage of animations in integration...

joshlory