CLI parameter --delay is ignored

[humdinger]

This is hrev36555.

The --delay parameter is ignored.

[anevilyak]

Just curious, you are also passing --silent, right? I had a quick glance and the code appears to be handling the delay parameter, but it's not used unless you're also asking for silent operation.

[humdinger]

You're right. With --silent it works. But it should also work without that, right? At least it did so some time ago, because my 'nifty' desklink script worked before:

#!/bin/sh

# Add Screenshot to the Deskbar tray

cd /boot/system/apps
desklink "cmd=Active window (2s):Screenshot --window --border --delay 2" "cmd=Remove replicant:desklink --remove=Screenshot" Screenshot

:)

[anevilyak]

As far as I can tell, without the silent parameter, all it does is set the starting delay that eventually shows up in the UI. No idea if that behavior was changed recently since I've never really kept up with Screenshot development. Without --silent I'm somewhat puzzled as to what you're expecting though, did you want it to wait the delay before popping up the screenshot window, or what?

[humdinger]

Yes. I want to get this 2 seconds delay to make my target window active, before the GUI pops up to set name, destination and save the bugger.

[Wim]

This could be the result of the changes I made to save the Screenshot settings. I'll look into it.

[Wim]

This patch makes sure that the the command line "--delay" switch is not ignored.

[Wim]

I patched the application in such a way that without specifying the delay on command line a screenshot with delay 0 is taken, even if the saved settings has a non-zero value.

[Wim]

Is there any chance to get this patch committed before the alpha2 release?

[Wim]

Fixed in hrev36969.