MapRandomizer icon indicating copy to clipboard operation
MapRandomizer copied to clipboard

No step change

Open change1331 opened this issue 9 months ago • 1 comments

Current step doesn't change when clicking on map icons. Ship icon flashes current step red if spoiler is on to indicate unclickable until f_DefeatedMotherBrain is reachable on current step.

change1331 avatar Apr 30 '25 22:04 change1331

This is a nice improvement. A couple things I notice that could use some refinement:

  • Clicking on a hidden item in the side panel (appearing as a chozo orb) pops up the route to the item without changing the selected step, which means the route can pass through rooms that aren't revealed. I think ideally, a hidden item in the side panel shouldn't have any reaction to hovering over it, and clicking on it should just select that step rather than opening an item route; i.e. it should behave the same as clicking elsewhere in that step's space in the side panel.
  • The behavior of the step flashing red doesn't seem intuitive to me. I think you could just make the Ship icon not have a reaction to hovering/clicking when the game is not yet logically beatable on the selected step. This would be consistent with the behavior of other icons on the map like logically unreachable items (chozo orbs) and flags. Alternatively, and possibly nicer, would be to have the tooltip display some info when hovering in all three of these cases, e.g. something like "Game not yet beatable on this step" (for the Ship), "Not reachable on this step" for unreachable items, "Not in logic on this step" for flags that aren't collectible. One-way reachable items already say "Not in logic on this step" in their tooltip, which seems nice, so it would seem natural to extend that kind of behavior to these other cases.

blkerby avatar Apr 30 '25 23:04 blkerby