Awesome: Was "scrot -s" bug fixed?

[initrider]

Hello. I am talking about this bug https://www.reddit.com/r/awesomewm/comments/7ktca8/hotkey_for_scrot_s_not_working_while_scrot_and/ - see answer (Edit 4)
I can also see this on archwiki https://wiki.archlinux.org/index.php/awesome#scrot:_Cannot_take_a_mouse_selected_screenshot_with_keyboard_shortcuts
Can you, please, give a comment?

[actionless]

it was always working for me

if something is not working for you: first start looking into awesome stderr/stdout

[actionless]

idk, mb because i am using spawn_with_shell instead of spawn and it's creating that additional delay to workaround the bug.. :-)

UPD: no, with normal spawn i neither can reproduce it

[psychon]

What exactly is the bug here? Edit 4 is:

xmonad and, apparently, awesomewm as well don't finish releasing the keyboard before spawning, so I changed my binding to

If you want to trigger a key binding when the key is released, just do so (notice the extra nil for the press argument to awful.key and instead of the third argument, I pass the callback function to the fourth argument, which is the argument named release):

awful.key({ "Shift", "Control"}, "Print", nil, function () awful.spawn("scrot -e 'mv $f ~/screenshots/'") end,
          {description = "take screenshot of all screens", group = "screen"}),

So, I'm tempted to call this a config error. If you consider "awesome's default config triggers keys on press instead of release"... well, dunno about that.

[psychon]

Could also some with an account there update the ArchLinux wiki to actually bind to release instead of using the sleep hack?

[alfunx]

@psychon: Done. https://wiki.archlinux.org/index.php/Awesome#scrot:_Cannot_take_a_mouse_selected_screenshot_with_keyboard_shortcuts

[actionless]

btw, are any of fullscreen advices there are actual? i think now all the browsers do that wm hints magic right to be fullscreen without defining extra rules (my wiki account is banned so i can't edit it)

[Elv13]

Closed due to need info being set and no reply from the author for months

[McSinyx]

@alfunx with updated API simply awful.spawn is enough I think.