mapeditor / tiled

Flexible level editor
https://www.mapeditor.org/
Other
11.26k stars 1.76k forks source link

Make Stamp Brush right-click cut instead of copying when Shift is held #3961

Closed kdx2a closed 5 months ago

kdx2a commented 5 months ago

Is your feature request related to a problem? Please describe. Moving a group of tiles on a tile layer is too convoluted for such a common operation. Unless I miss something, here's the fastest way to do so (with default keybindings):

Describe the solution you'd like Allow to cut a rectangular area in Stamp Brush mode by holding Shift and right-clicking (in addition to the current copying behavior).

Describe alternatives you've considered

eishiya commented 5 months ago

Displacing tiles within a layer is a common operation for some users, but is never needed for others, so overall, it's a rarely-needed feature, which is why it's not gotten much attention.

I think your proposed expansion of the current copying behaviour sounds great! Way more elegant than the more complex behaviour I was thinking of. Shift+right-click seems to be free, it currently causes the Brush Stamp to just not do anything.

Big fat +1 to making Stamp Brush right-click cut instead of copying when Shift is held. (For clarity, consider changing your issue title to something that describes your proposed change, as your title makes it sound like you want something more complicated.)

I'd previously scripted a Move Tiles Tool that allows moving tiles in multiple drags and via the arrow keys, and while a more complex style of operation like that can be desirable, in practice, a quicker way to just cut and paste would be way faster, even in scenarios where small, precise movements of large groups of tiles are desired.

kdx2a commented 5 months ago

Thank you for your feedback @eishiya! I had time off today, so I looked into the codebase to see how possible it would be and I ended up implementing the feature myself. It wasn't as straightforward as I would have expected, but it works very well now and is very convenient in terms of workflow.

bjorn commented 5 months ago

Implemented by #3963.