View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0029126||2.x IDE Bugs||Room Editor||Public||2018-03-12 12:22||2018-09-28 14:08|
|Reporter||Sam Cumming||Assigned To||CoreTech|
|Priority||Medium||Severity||C - General||Reproducibility||100%|
|Target Version||Fixed in Version|
|Summary||0029126: Room Editor: Changing Room size will remove tiles from the edges of the room, undo doesn't recover them|
|Description||When you change the size of a room to make it smaller, any tiles that you have that now fall outside of the room boundaries are deleted. However, if you press Undo, the room will return to the previous size, but the tiles removed will not be recovered.|
Expected: When you press Undo, all instances, tiles and assets that were removed are recovered in their original positions.
|Steps To Reproduce||1. Open any project.|
2. Open a Room, add some assets to it. Make sure the edges of the room have assets.
3. Make the room smaller, observe that assets outside of the boundary are removed.
5. Observe that the room is correctly resized, observe that assets are not recovered.
|2.x IDE Found In||126.96.36.1993|
|2.x Runtime Found In||188.8.131.52|
|2.x IDE Verified In|
|2.x Runtime Verified In|
||I would expect that we showed you a "There are resources outside the new room size - these will be lost. Are you sure you want to make the room smaller?" dialog first, regardless. If we don't, add this as a separate bug, please.|
|It should either "undo", or show a warning that it's not-undoable. Can also add the "do not show again" message to the dialog.|
Last edited: 2018-03-23 15:53
The undo does happen if you click outside of the textbox and undo there. If you undo within the textbox until it displays the original number and then hit enter this counts as entering a new value for the room undo history which is not the same as undoing.
Could possibly do something where undoing to the point of the last 'stored' value triggers an 'undo' in the containing undo context, would then also need to do something similar for redo. This would be a general case for all mvc bound textboxes not just this instance.