]> git.mdlowis.com Git - proto/labwc.git/commit
fix interaction between Move action and click bindings
authorbi4k8 <bi4k8@github>
Thu, 2 Dec 2021 14:07:17 +0000 (14:07 +0000)
committerJohan Malm <johanmalm@users.noreply.github.com>
Thu, 2 Dec 2021 17:58:31 +0000 (17:58 +0000)
commita94fb3b225e3e9846678dd76c5ed905fde99fd39
tree6cfcda94d153cab25ee443bea98dc405edccc8e5
parentd8bbb751d20af2c36123c6ea6bc99a69bffd2ca0
fix interaction between Move action and click bindings

in order to allow the Move action to be bound to "press" on mouse binding contexts that also have "click" bindings, we should not short-circuit event processing when the input_mode is not passthrough (the "normal" mode). doing so seems to have been intended to prevent mouse bindings from being triggered during move/resize/menu interactions, but this does not seem to occur in practice. instead, `cursor_button`'s early return in this case caused the "release" side of "click" bindings to be ignored if their "press" side began an action that changes input_mode (e.g. Move).

the cleaner way to fix this interaction would be to use "drag" rather than "press" for Move bindings, but implementing "drag" is more complexity than I want to include in this changeset.

this change is its own commit to make it easy to bisect for, in case it causes regressions.
src/cursor.c