Bug 423442

Summary: Freely-resizable version of the Media widget has a suboptimal layout for low width-to-height aspect ratio
Product: [Plasma] plasmashell Reporter: Till Schäfer <till2.schaefer>
Component: Media Player widgetAssignee: Kai Uwe Broulik <kde>
Status: RESOLVED FIXED    
Severity: minor CC: nate, plasma-bugs-null, uhhadd
Priority: NOR Keywords: usability
Version First Reported In: 5.19.1   
Target Milestone: 1.0   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:
Attachments: screenshot of media player widget with low with to height aspect ratio

Description Till Schäfer 2020-06-24 12:13:51 UTC
Created attachment 129633 [details]
screenshot of media player widget with low with to height aspect ratio

The media player widget has a very bad space utilization when using a low width to height aspect ratio (see attached screenshot). The text is always shown on the side of the image. However, in such situations it would be much better to show the text below the image and make the image larger. 

Thus, this is a suggestion to dynamically adjust the layout (text beside or below the image) based on the aspect ration of the widget.


Operating System: Gentoo Linux
KDE Plasma Version: 5.19.1
KDE Frameworks Version: 5.71.0
Qt Version: 5.15.0
Kernel Version: 5.7.5-gentoo
OS Type: 64-bit
Processors: 8 × Intel® Core™ i7-4790K CPU @ 4.00GHz
Memory: 23.5 GiB of RAM
Graphics Processor: GeForce GTX 970/PCIe/SSE2
Comment 1 Nate Graham 2020-06-25 18:50:13 UTC
We had the exact same issue in Elisa recently, and fixed it there by moving the text to below the artwork when the view is more vertical than horizontal. See https://invent.kde.org/multimedia/elisa/-/commit/6f48964505a3c11839a4aad196f492b4521fdee3

I wonder if we should extract the Elisa view into a shared media player art view that can live in Kirigami and knows how to switch from a vertical to a square or horizontal layout, and then use that thing both here and in Elisa.

Thoughts, Carson?
Comment 2 Till Schäfer 2020-06-25 21:37:09 UTC
Some time ago I have also created some mockups for playbar2, another nice media widget, which had the same problems. Playbar ended up in three different layouts: vertical, balanced, horizontal. 

https://github.com/audoban/playbar2/issues/16

Maybe you can use some of the ideas...
Comment 3 Till Schäfer 2023-06-30 14:22:24 UTC
Somebody seem to have implemented it in the meanwhile. Text is now displayed below the image for small widths.