gabm / Satty

Satty - Modern Screenshot Annotation. A tool inspired by Swappy and Flameshot.
Mozilla Public License 2.0
486 stars 22 forks source link

Satty doesn't seem to work with grimblast (Hyprland utility) #66

Closed UnixLudi0 closed 6 months ago

UnixLudi0 commented 6 months ago

As I know grimblast using grim and slurp, so I don't know why this happen: Maybe I'm doing something wrong? image image image

gabm commented 6 months ago

apparently grimblast does produce a file format that satty can't read. I am a bit surprised, but lets have a look.. can you send me a sample?

grimblast copy screen - > outputfile

gabm commented 6 months ago

I just had a look at the source code of grimblast... you are using it wrong.

The help for copy states: "Copy the screenshot data into the clipboard.". That is not what you want! You want to use save: " "Save the screenshot to a regular file or '-' to pipe to STDOUT.". So your fixed command is:

grimblast save - | satty -f -

UnixLudi0 commented 6 months ago

Nice 0 byte size image That's still not work image

ShadowS0ng commented 6 months ago

grimblast save area - | satty --filename - it works without problems for me

taylorcoffelt commented 6 months ago

Interesting, I'm getting the following with that:

$ grimblast save area - | satty --filename -
[destroyed object]: error 0: Unsupported format
failed to screenshoot all outputs
Error: Unrecognized image file format
Error: Unrecognized image file format

Looks more like a grim / grimblast issue:

$ grim -g "$(slurp -o -r -c '#ff0000ff')" -
[destroyed object]: error 0: Unsupported format
failed to screenshoot all outputs

$ grim
[destroyed object]: error 0: Unsupported format
failed to screenshoot all outputs

Edit: https://github.com/hyprwm/Hyprland/issues/4735 Turning off 10-bit color fixes the issue for me

gabm commented 6 months ago

looks like .. maybe we can support them but opening an issue there - or better - fixing it?