Editor: Release mouse button does not work when placing things and mouse gets under a window

[kaputtnik]

Steps to reproduce:

1. Open editor
2. Open Tools window and immovables window
3. Select an immovable (f.e. the column "ruin") and don't close the immovable window
4. Hold down left mouse button and place columns on the map
5. Move the mouse under the Immovables window (while hold down the mouse button) and release the mouse button
6. Move the mouse and everywhere the mouse is moved to, columns (ruins) will be placed, even the mouse button is released.
7. Select another immovable or open terrain menu and select a terrain
8. Now the mouse is placing the thing you selected on each place where the mouse moves to.

To stop this you have to click once again on the map.

This happens if the mouse pointer gets under each open window.

Even worth: If you select the "Set origin" button after releasing the mouse button under a window, each time the mouse moves, a click event is triggered and the origin is calculated each time the mouse is over a node of the map.

I guess this is no regression, because i noticed this behavior long time ago.

[GunChleoc (gunchleoc)]

I can reproduce this with Build 18 already. Probably related to Windows having the topcaller property.

[3plus4i (tobi-heinz)]

This is not related to a specific OS, I'm affected on Xubuntu 16.04 (and formerly 12.10) as well.

Though I don't actually regard it as a bug but a feature. I use it regularly to "paint" big areas. It's also easy to fix (by clicking in the main window) and, if necessary, can be undone without trouble. So does it really have to be fixed?

[GunChleoc (gunchleoc)]

Sorry, I meant windows, not Windows in #1. Yes, it is OS-independent.

[kaputtnik (franku)]

> Though I don't actually regard it as a bug but a feature. I use it regularly to "paint" big areas. It's also easy to fix (by clicking in the main window) and, if necessary, can be undone without trouble. So does it really have to be fixed?

I am always confused about this behavior. At least i think releasing a mouse button should always behave the same: The current action should end :-)

[3plus4i (tobi-heinz)]

I can't say that you don't have a point there. So I guess I'm neutral on whether to change this or not.

[GunChleoc (gunchleoc)]

Looks like a bug in UI::Window to me - see linked branch & Merge Request.

[GunChleoc (gunchleoc)]

Fixed in build20-rc1