The problem was that the earlier "SwEditWin: disable map mode in
LogicMouseButtonUp/Down" commit disabled map mode only during the
handling of the mouse event, so in case a method invoked by the idle
timer used our logic coordinates later, it still performed the pixel ->
logic conversion, leading to jumping cursor after a mouse click.

Fix this by disabling map mode early, even outside the mouse handlers.

Change-Id: Ib9f194f5555536a9af21a2d66bd7345addee72c3