aboutsummaryrefslogtreecommitdiff
path: root/TODO
diff options
context:
space:
mode:
authorAvi Halachmi (:avih) <avihpit@yahoo.com>2020-04-02 11:43:22 +0300
committerHiltjo Posthuma <hiltjo@codemadness.org>2020-04-02 14:41:03 +0200
commit28ad28839985e965c9ca06a9a202523414c84ac4 (patch)
treeb5e83f2b48c180743f34296251fbe1a863575382 /TODO
parent51e19ea11dd42eefed1ca136ee3f6be975f618b1 (diff)
downloadst-28ad28839985e965c9ca06a9a202523414c84ac4.tar.gz
st-28ad28839985e965c9ca06a9a202523414c84ac4.tar.bz2
mouseshortcuts: fix custom modifier on release
This line didn't work at mshortcuts at config.h: /* mask button function arg release */ { ShiftMask, Button2, selpaste, {.i = 0}, 1 }, and now it does work. The issue was that XButtonEvent.state is "the logical state ... just prior to the event", which means that on release the state has the Button2Mask bit set because button2 was down just before it was released. The issue didn't manifest with the default shift + middle-click on release (to override mouse mode) because its specified modifier is XK_ANY_MOD, at which case match(...) ignores any specific bits and simply returns true. The issue also doesn't manifest on press, because prior to the event Button<N> was not down and its mask bit is not set. Fix by filtering out the mask of the button which we're currently matching. We could have said "well, that's how button events behave, you should use ShiftMask|Button2Mask for release", but this both not obvious to figure out, and specifically here always filtering does not prevent configuring any useful modifiers combination. So it's a win-win.
Diffstat (limited to 'TODO')
0 files changed, 0 insertions, 0 deletions