Suggestion if you want people to look at your code (which I'm assuming you do since you linked to the PR): make whitespace changes in a separate commit from functional changes.[0]<p>This PR is a 50-line diff, but only about 8 lines are <i>real</i> changes. The rest are just whitespace changes that seem to have nothing to do with the feature you added.<p>IsoCity's neat. My suggestion for the functionality would be to make it easier for the user to join roads together. I drew roads at a 90-degree angle expecting the game to auto-join them, but it turns out I have to go find the right 90-degree tile to do it manually.<p>[0] <a href="https://mtlynch.io/code-review-love/#6-separate-functional-and-non-functional-changes" rel="nofollow">https://mtlynch.io/code-review-love/#6-separate-functional-a...</a>
Neat, the URL uniquely encodes the map state. Here's a maze I made:<p><a href="https://victorribeiro.com/isocity/#KQIXEikNDSYlGCQoKScpHB0pFiMDJhYnJgICKCklAyQCAicUGSYFJyIPJg0NDSgJKA==" rel="nofollow">https://victorribeiro.com/isocity/#KQIXEikNDSYlGCQoKScpHB0pF...</a>
Yeah, it's a Friday night and I don't have anything fun to do, so I have just added undo and redo on IsoCity [1]. Hope you like it<p>1 - <a href="https://victorribeiro.com/isocity/" rel="nofollow">https://victorribeiro.com/isocity/</a>