Bug 444204

Summary: Inhibition reason strings are not user-friendly
Product: [Unmaintained] Powerdevil Reporter: ratijas <me>
Component: generalAssignee: Plasma Bugs List <plasma-bugs-null>
Status: RESOLVED DOWNSTREAM    
Severity: normal CC: nate
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: Battery notes inhibition firefox video-playing

Description ratijas 2021-10-21 16:12:53 UTC
Created attachment 142730 [details]
Battery notes inhibition firefox video-playing

SUMMARY
Inhibition reasons are not designed to be user-facing? Or at least they are mis-used by app developers.

STEPS TO REPRODUCE
1. Play video in VLC
2. Play video in firefox

OBSERVED RESULT
Firefox reports inhibition reason as "audio-playing".
VLC reports localized "Воспроизведение мультимедиа." with a dot at the end.

Both look poor, for example, in the battery applet of Plasma. Especially the dot at the end, since it displayed in (parenthesis.)

EXPECTED RESULT
Inhibition reasons should be uniform, localized, and user-friendly.

SOFTWARE/OS VERSIONS
Operating System: Arch Linux
KDE Plasma Version: 5.23.80
KDE Frameworks Version: 5.88.0
Qt Version: 5.15.2
Kernel Version: 5.14.12-arch1-1 (64-bit)
Graphics Platform: X11
Processors: 8 × Intel® Core™ i7-6700HQ CPU @ 2.60GHz
Memory: 15.6 GiB of RAM
Graphics Processor: NVIDIA GeForce GTX 970M/PCIe/SSE2

ADDITIONAL INFORMATION
The best we could do is probably contact app developers one by one?
Comment 1 Nate Graham 2021-10-21 17:25:11 UTC
Yeah since the text comes from the app itself, when you come across an app that sets a bad string, you need to report it to them. Not all apps set stupid text here, and the ones that do need to be fixed. Thanks!
Comment 2 ratijas 2021-10-21 22:21:44 UTC
I feel so tired just from filing them in, ugghh...

https://code.videolan.org/videolan/vlc/-/issues/26219
https://bugzilla.mozilla.org/show_bug.cgi?id=1737171

Turns out, Elisa music player is doing better them both of those, but still not perfect because too many Capital Letters for No Reason. Everybody has their own style. I bet it was never documented in standard how that "reason" should look like.
Comment 3 Nate Graham 2021-10-21 22:49:37 UTC
I would imagine not, yeah. FWIW I've fixed the unnecessary Title Case in Elisa with https://invent.kde.org/multimedia/elisa/commit/bc32cc3e1f23f26c76462dd36ab749f7cc75f03e.