Skip to content
This repository has been archived by the owner on Mar 30, 2022. It is now read-only.

Consider generating the output filename if none is specified #10

Closed
emersion opened this issue May 20, 2018 · 7 comments
Closed

Consider generating the output filename if none is specified #10

emersion opened this issue May 20, 2018 · 7 comments
Labels
enhancement New feature or request

Comments

@emersion
Copy link
Owner

No description provided.

@emersion emersion added the enhancement New feature or request label May 20, 2018
@emersion emersion changed the title Consider generating default filenames Consider generating the output filename if none is specified May 20, 2018
@emersion
Copy link
Owner Author

emersion commented Jun 8, 2018

Maybe save it in $XDG_PICTURES_DIR instead of the current directory? (With an override like $GRIM_DEFAULT_DIR)

@emersion emersion added the question Further information is requested label Jul 13, 2018
@gagath
Copy link

gagath commented Dec 2, 2018

Here is the format that scrot uses for autogenerated filenames that could be reused in grim:

 2018-12-02-103517_1920x1080_scrot.png

@ghost
Copy link

ghost commented Dec 23, 2018

maybe output to stdout? that way it is easier to pipe to wl-copy, and redirecting with > is still easy.
also it would allow for something like tee to output to /tmp/screenshot.png AND to the clipboard.

@emersion
Copy link
Owner Author

No, it would be too easy to get junk in the terminal this way. grim supports - to pipe its output.

@ghost
Copy link

ghost commented Dec 23, 2018

@emersion it would be good to note in the usage (-h) or really everywhere else

@emersion
Copy link
Owner Author

emersion commented Dec 23, 2018

It's documented in the manpage: https://github.com/emersion/grim/blob/master/grim.1.scd#L17

I'm willing to add a "copy to clipboard" example to the README once this pull request gets merged.

@emersion emersion removed the question Further information is requested label Dec 31, 2018
@ammgws
Copy link
Contributor

ammgws commented Dec 16, 2019

@emersion This can be closed now :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants