Bug 470065

Summary:	Screenshot variable (format) is passed to mpv incorrectly.
Product:	[Applications] Haruna	Reporter:	jacky_009
Component:	generic	Assignee:	george fb <georgefb899>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	NOR
Version First Reported In:	0.11.0
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

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