scrot saves to $HOME rather than $HOME/Pictures by default

Bug #1732871 reported by Pander
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
lubuntu-default-settings (Ubuntu)
Won't Fix
Low
Unassigned

Bug Description

Two things to fix here:

 1. Change A-Print and Print in OpenBox config to call `lxsession-default screenshot_manager`
 2. Change LXSession screenshot_manager to something like `scrot '%Y-%m-%d_$wx$h.png' -e 'mv $f ~/Pictures`

Tags: bionic
Revision history for this message
ԜаӀtеr Ⅼарсһуnѕkі (wxl) wrote : Re: [Bug 1732871] [NEW] Ship with screen shot application

it's already there. `man scrot`. now why the print screen doesn't work
is another story. maybe file a new bug?

Revision history for this message
Pander (pander) wrote : Re: Ship with screen shot application

Ah, found them. The remaining bug I would like to report is that the screen shots are saved by default to ~/Pictures as is the case in other tastes of Ubuntu. Advantage is that it conforms to other Ubuntu tastes, user expectation and that it get picked up by image viewing and image manipulation applications working that directory and backup monitoring that directory.

Revision history for this message
Pander (pander) wrote :
Revision history for this message
ԜаӀtеr Ⅼарсһуnѕkі (wxl) wrote : Re: [Bug 1732871] Re: Ship with screen shot application

Yeah, that's a reasonable thing to consider. Unfortunately, `scrot`
provides no configuration options. It's certainly possible to add the
suggestions to the hotkey command, but even `man scrot` suggests doing
something like `scrot '%Y-%m-%d_$wx$h.png' -e 'mv $f ~/Pictures'` which
is a pretty clear message that they probably have little interest in
changing that.

In fact, if [this repo][1] is the upstream code (I have reason to
believe it is), at least the person replying to [this issue][2] makes it
pretty darn clear that that's the way it's supposed to function and
that's the way it will remain.

That said, I think there's little that we can do except change the
command for the hotkey.

[1]: https://github.com/dreamer/scrot
[2]: https://github.com/dreamer/scrot/issues/4

P.S. if you dig up the debian/watch file it has a fake entry apparently
meant to indicate that there is no current upstream site for the
package. I'm not sure if Tom Gilbert decided to quit developing or not,
but his website is there and the /scrot folder is not. The initial
commit on that repo suggests that it was done with Tom, so maybe he
passed the torch. Hasn't been any development 2003-2010, so I guess that
makes sense.

Revision history for this message
Pander (pander) wrote : Re: Ship with screen shot application

Thanks. I have tried adding options to ~/.config/lxsession/Lubuntu/desktop for the line with scrot but without succes. What is the best way to configure this for a complete system and not for users individually via an alias?

Revision history for this message
ԜаӀtеr Ⅼарсһуnѕkі (wxl) wrote : Re: [Bug 1732871] Re: Ship with screen shot application

The global files are all in /etc/xdg. Note that those are not
necessarily guaranteed to persist across version updates.

Revision history for this message
Pander (pander) wrote : Re: Ship with screen shot application

Which lines need to get altered? only 250 and 317 in /etc/xdg/openbox/rc.xml ?

$ grep scrot -A1 -B1 -nrI /etc/xdg
/etc/xdg/lxsession/Lubuntu/desktop.conf-36-
/etc/xdg/lxsession/Lubuntu/desktop.conf:37:screenshot_manager/command=scrot
/etc/xdg/lxsession/Lubuntu/desktop.conf-38-
--
/etc/xdg/lxsession/Lubuntu-Netbook/desktop.conf-35-
/etc/xdg/lxsession/Lubuntu-Netbook/desktop.conf:36:screenshot_manager/command=scrot
/etc/xdg/lxsession/Lubuntu-Netbook/desktop.conf-37-
--
/etc/xdg/openbox/rc.xml-247- </keybind>
/etc/xdg/openbox/rc.xml:248: <!-- Take a screenshot of the current window with scrot when Alt+Print are pressed -->
/etc/xdg/openbox/rc.xml-249- <keybind key="A-Print">
/etc/xdg/openbox/rc.xml:250: <action name="Execute"><command>scrot -s</command></action>
/etc/xdg/openbox/rc.xml-251- </keybind>
--
/etc/xdg/openbox/rc.xml-314- </keybind>
/etc/xdg/openbox/rc.xml:315: <!-- Launch scrot when Print is pressed -->
/etc/xdg/openbox/rc.xml-316- <keybind key="Print">
/etc/xdg/openbox/rc.xml:317: <action name="Execute"><command>scrot</command></action>
/etc/xdg/openbox/rc.xml-318- </keybind>

Good if they are overwritten with version updates, as I hope this setting can become default for Lubuntu (which is inline with other flavors of Ubuntu).

Revision history for this message
ԜаӀtеr Ⅼарсһуnѕkі (wxl) wrote :

The lxsession option controls the default screen shot app, which anything else could refer to and then the screen shot application wouldn't need to be hard-coded everywhere.

The OpenBox option controls the response to the hot keys specified.

So probably both.

We'll consider adding that as a default option.

By the way, did you file a bug for the print screen not working or did it actually work?

Revision history for this message
Pander (pander) wrote : Re: Ship with screen shot application

It did work, didn't find them initially. They were in home and not in ~/Pictures where all Ubuntu flavors store screenshots.

tags: added: bionic
Pander (pander)
tags: added: cosmic
Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in lubuntu-default-settings (Ubuntu):
status: New → Confirmed
affects: lubuntu-tweaks → lubuntu-default-settings (Ubuntu)
Changed in lubuntu-default-settings (Ubuntu):
importance: Undecided → Low
status: New → Triaged
summary: - Ship with screen shot application
+ scrot saves to $HOME rather than $HOME/Pictures by default
description: updated
Pander (pander)
tags: added: disco
removed: bionic cosmic
Revision history for this message
ԜаӀtеr Ⅼарсһуnѕkі (wxl) wrote :

If we're considering this a bug against lubuntu-default-settings, then it does not affect cosmic or disco, as Lubuntu no longer uses scrot or, for that matter, LXDE. If you're trying to continue to use LXDE in cosmic or disco, that's fine, but since the fix involves dealing with packages outside of scrot itself, you'll have to look elsewhere for the resolution.

tags: added: bionic
removed: disco
Revision history for this message
ԜаӀtеr Ⅼарсһуnѕkі (wxl) wrote :

Lubuntu doesn't use LXDE anymore so dropping lubuntu-default-settings.

Changed in lubuntu-default-settings (Ubuntu):
status: Triaged → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.