search

graememcc / micropolisJS

A port of Micropolis to JS/HTML5
Other
639 stars 94 forks source link

Fix placing railroad tiles #36

Closed AnnikaCodes closed 2 years ago

AnnikaCodes commented 2 years ago

I wasn't able to properly test this locally because I couldn't install dependencies. Which Node.js version do you use for this?

However, I tested this by modifying the minified JS directly on my fork, and confirmed that this fixes placing railroad tiles.

Fixes #35.

graememcc commented 2 years ago

Thanks for the fix!

Looks like the machine I have the repo on currently has 14.17.6 - suspect the contents of node_modules predate that, unsure what version I was on when I last installed dependencies. I'll take a pass over the dependencies at some point.