Josh Lory

Results 37 issues of Josh Lory

Resurrect the changes in [PR #529](https://github.com/code-dot-org/craft/pull/529/files#diff-894dfbadc994764b5b510d470f495021L675) to draw the underwater overlay without the WebGL shader. Not clear yet if the shader is causing issues on low-end Chromebooks, but creating this...

I don't think we'll want to merge this until we _also_ ship the unbuild files to npm. Otherwise we'll double-include the `babel-polyfill` when importing the built `dist/main.js` in the main...

I've selected for refactoring 37 lines of code which are duplicated in 3 file(s) ([1](https://github.com/code-dot-org/craft/blob/9db1d6c80e8832e06ef0fdef4e092038b0a6b47a/src/js/game/Entities/Creeper.js#L33-L76), [2](https://github.com/code-dot-org/craft/blob/9db1d6c80e8832e06ef0fdef4e092038b0a6b47a/src/js/game/Entities/IronGolem.js#L32-L75), [3](https://github.com/code-dot-org/craft/blob/9db1d6c80e8832e06ef0fdef4e092038b0a6b47a/src/js/game/Entities/Zombie.js#L99-L142)). Addressing this will make our codebase more maintainable and improve [Better Code Hub](https://bettercodehub.com)'s...

cleanup

`gameControllerConfig.Phaser` is only used in GameController.js: https://github.com/code-dot-org/craft/blob/3c2fc9506136fd32164ea8c3cadd826acb53a07e/src/js/game/GameController.js#L45 But all of LevelView.js still references the global `window.Phaser`: https://github.com/code-dot-org/craft/blob/3c2fc9506136fd32164ea8c3cadd826acb53a07e/src/js/game/LevelMVC/LevelView.js#L30

cleanup

Ref: https://github.com/code-dot-org/craft/blob/70e7f4f09f6ee9bc2ce011f89b5ec28f36fab949/demo/js/phaser.js

cleanup

Expected behavior: ![steve_boat](https://user-images.githubusercontent.com/413693/47175985-5cae8200-d2c9-11e8-860c-558262858ae1.gif) Actual: ![image](https://user-images.githubusercontent.com/413693/47175936-3d175980-d2c9-11e8-8e9f-779f93748342.png)

help wanted

The added aquatic 01-03 mp3 files are much larger than previous years. We'll also need the .ogg version for our sound player. https://github.com/code-dot-org/craft/blob/5a8b2378b4bfb0e7d9d68de4b768ca947455e9df/src/assets/audio/ogg2mp3#L1-L2 ![image](https://user-images.githubusercontent.com/413693/47758604-f894a200-dc68-11e8-897b-53be004a3759.png) The nether music from last year...

Why are there two (that aren't always set consistently)?

cleanup

See https://github.com/code-dot-org/craft/issues/374#issuecomment-339499640. We've been approximating this with the "Invisible" block type, but this is fragile and breaks when we import new versions of Blocks.json. https://github.com/code-dot-org/craft/blob/ffb56b9281db1b3201c1e866856f12e319fd4ba9/src/js/game/LevelMVC/LevelBlock.js#L77-L82 On October 29, 2018 I...

help wanted

See https://github.com/code-dot-org/craft/pull/507#pullrequestreview-165340120. We think this is a remnant from the first year, where action plane blocks were collideable, but decoration plane blocks were not.

cleanup