Page 1 of 1

Greedy undo does not work for tiles

Posted: Sun Feb 23, 2025 2:06 pm
by Hares
126636: Version 2.0.34 introduced new feature - greedy undo.
Steps to Reproduce
  1. Select any tile-placing tool
  2. Drag it (either manually or via planner)
  3. Undo the last action
Expected:
  • All dragging action is undone
Actual:
  • Only the last iteration of dragging is undone
  • This is inconsistent with greedy undo for dragging items (belts, powerpoles, etc)
Click to expand the recording

P.S.
I still think greedy undo should be on separate hotkey (see discussion), but the inconsistency with tile placement is definitely a bug. Either both should be greedy or both should be non-greedy.

P.P.S.
I reported this bug in 2.0.34 release notes discussion, but it went unnoticed.

Edit
Version prefix in the topic name changed from 2.0.34 to 2.0.35 to match the version this is report is recorded on. (Even though first discovered and first reported on 2.0.34.)

Re: [2.0.35] Greedy undo does not work for tiles

Posted: Thu Feb 27, 2025 5:27 pm
by Hares
More on topic. Dragging power poles is also undone one-by-one which is also inconsistent but perfectly fine for me.

Re: [2.0.35] Greedy undo does not work for tiles

Posted: Thu Feb 27, 2025 5:28 pm
by boskid
Hares wrote: Thu Feb 27, 2025 5:27 pm More on topic. Dragging power poles is also undone one-by-one which is also inconsistent but perfectly fine for me.
Dragging poles is an explicit exception of this system. Tiles are simply not implemented.

Re: [2.0.35] Greedy undo does not work for tiles

Posted: Mon Mar 03, 2025 2:55 pm
by boskid
Thanks for the report. I am considering this to be a feature request as undo merging was only implemented for entities.