b4n left a comment (geany/geany-plugins#1458) OK, looks good now :+1:
Just 2 more things… yeah. * putting `Key_Released_CallBack()` in its original position minimizes the overall diff. Not very important though -- I mostly moved it back to check how different it actually ended up being. * More importantly, for me some combinations result in weird double actions (e.g. <kbd>Ctrl+-</kbd> that also zooms out, and <kbd>Ctrl+0</kbd> that also resets zoom). This is due to the fact that the plugin for some reason work on key releases, and most of other code uses key presses, so one action is performed upon press, another upon release. I'd think it makes more sense to consume the press as well to avoid these double effects. What do you think? FWIW, I added those 2 to https://github.com/b4n/geany-plugins/commits/num_bkm/ in case it makes sense to add here. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1458#issuecomment-3025290838 You are receiving this because you are subscribed to this thread. Message ID: <geany/geany-plugins/pull/1458/[email protected]>
