search

jellyfin / jellyfin-media-player

Jellyfin Desktop Client
GNU General Public License v2.0
3.21k stars 321 forks source link

Add player screenshots #532

Open HamletDuFromage opened 11 months ago

HamletDuFromage commented 11 months ago

Is your feature request related to a problem? Please describe. It'd be nice to be able to take a screenshot at the press of a keybind, and have it saved with sane (or user configurable) paths such as filename + date

Describe the solution you'd like Press a key (s?) to take a screenshot. Use MPV's syntax in the options to specify where to save it (screenshot-template="~/Pictures/jellyfin/%F/%F_%p__%tm-%tY")

Describe alternatives you've considered An icon on the player could also do the trick. Taking a picture using whatever built-in tool is doable, but less elegant, more cumbersome.

Additional context N/A

adamboulton24 commented 2 months ago

Considering the app uses mpv as the base for it's media player, and mpv has 's' set to capture a screenshot of the media being displayed, I can't imagine why this feature wasn't carried over.

I'm still really hoping for this to be added at some point.

pearcemichal commented 2 months ago

My primary display is an ultra-wide monitor, which means that video media is always going to have black bars on either side, which is why print screen doesnt work for me, and manually taking a screen snip is annoying, and will result in images always being a few pixels off in size.

Further, manually saving / organizing screenshots can be a pain when you have a lot of them. By building the screenshot tool into Jellyfin, it would allow for automatically saving screenshots into folders by show name / episode (configurable)

As an additional enhancement on top of this, it would be very cool if a section were added to the show page that would allow you to display and browse user screenshots of that show. Something to think about if image gallery support is ever added.