[HN Gopher] Show HN: I've added undo and redo to IsoCity
       ___________________________________________________________________
        
       Show HN: I've added undo and redo to IsoCity
        
       Author : atum47
       Score  : 20 points
       Date   : 2024-10-12 02:43 UTC (1 days ago)
        
 (HTM) web link (github.com)
 (TXT) w3m dump (github.com)
        
       | atum47 wrote:
       | 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
       | 
       | 1 - https://victorribeiro.com/isocity/
        
       | mtlynch wrote:
       | 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]
       | 
       | This PR is a 50-line diff, but only about 8 lines are _real_
       | changes. The rest are just whitespace changes that seem to have
       | nothing to do with the feature you added.
       | 
       | 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.
       | 
       | [0] https://mtlynch.io/code-review-love/#6-separate-
       | functional-a...
        
         | elliottcarlson wrote:
         | You can also add ?w=1 to a PR to hide whitespace related
         | changes; i.e.
         | https://github.com/victorqribeiro/isocity/pull/26/files?w=1
        
           | gurjeet wrote:
           | There's a UI element for it now. Click on the gear icon and
           | check the 'Hide whitespace' checkbox, then reload.
        
         | ianbicking wrote:
         | A UI that might be relatively easy for the road problem: when
         | you right click on an empty tile, in the left hand tile
         | selector show all the tiles that would be "valid" in that
         | position. You'd have to add metadata about how tiles hook up,
         | but you wouldn't need to get all fancy about constraints and
         | stuff, or picking from several valid options.
        
         | playingalong wrote:
         | Re whitespace: ... or use the w=1 hidden query parameter:
         | https://github.com/victorqribeiro/isocity/pull/26/files?w=1
        
         | atum47 wrote:
         | I did push two commits, I think they got squashed on the merge
        
       | paulgb wrote:
       | Neat, the URL uniquely encodes the map state. Here's a maze I
       | made:
       | 
       | https://victorribeiro.com/isocity/#KQIXEikNDSYlGCQoKScpHB0pF...
        
       ___________________________________________________________________
       (page generated 2024-10-13 22:00 UTC)