Bug 470065 - Screenshot variable (format) is passed to mpv incorrectly.
Summary: Screenshot variable (format) is passed to mpv incorrectly.
Status: RESOLVED FIXED
Alias: None
Product: Haruna
Classification: Applications
Component: generic (other bugs)
Version First Reported In: 0.11.0
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: george fb
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-05-20 21:50 UTC by jacky_009
Modified: 2023-05-20 22:09 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description jacky_009 2023-05-20 21:50:28 UTC
SUMMARY
***
Haruna can save screenshots as jpg only. Something other than default is not accepted.
***


STEPS TO REPRODUCE
Go to the video settings, there under Format for screenshots the variables are in upper case. They are also written into the config and used with MPV. But MPV uses lowercase letters. So instead of PNG the png. Only if you manually set to "png" in the config file, screenshots are also saved as png because mpv can recognize the variable again.

OBSERVED RESULT
Screenshots are saved as jpg only.

EXPECTED RESULT
The correct variables should be passed to MPV so that PNG, WebP etc. are also accepted.

SOFTWARE/OS VERSIONS
Operating System: Manjaro Linux 
KDE Plasma Version: 5.27.4
KDE Frameworks Version: 5.105.0
Qt Version: 5.15.9
Kernel Version: 6.1.26-1-MANJARO (64-bit)
Graphics Platform: X11

ADDITIONAL INFORMATION
Fix is provided, it just needs to be written correctly in lower case in the config file for Haruna.
Comment 1 jacky_009 2023-05-20 22:09:37 UTC
sorry, my mistake. for whatever reason still started version 0.10 instead of 0.11