Bug 390280

Summary: Media Player's UI is too big
Product: [Plasma] plasmashell Reporter: Tania Raven <auto>
Component: Media PlayerAssignee: Kai Uwe Broulik <kde>
Severity: wishlist CC: anditosan1000, bugseforuns, kde, nate, plasma-bugs
Priority: NOR Keywords: junior-jobs
Version: 5.12.0Flags: kde: VisualDesign+
Target Milestone: 1.0   
Platform: Neon   
OS: Linux   
Latest Commit: Version Fixed In:
Attachments: A screenshot of the media player applet on Neon (1920x1080)

Description Tania Raven 2018-02-11 17:38:48 UTC
Created attachment 110545 [details]
A screenshot of the media player applet on Neon (1920x1080)

The controls in the media player applet in the Plasma panel look unnecessarily large. The entire applet could be shrunk or more buttons could be added. The padding around the edges is too thin as well, making this UI look unprofessional.

The lock screen media controls are much better - maybe these could be used?
Comment 1 David Edmundson 2018-02-11 21:41:02 UTC
Tagging VDG to state a resolution, though it's obviously not always possible to satisfy everyone's personal preferences when it comes to design
Comment 2 Andres Betts 2018-02-11 21:45:19 UTC
Kai works on some of this stuff. I do agree that there is a lot of wasted space in that screenshot. Kai? What do you think?
Comment 3 Kai Uwe Broulik 2018-02-13 08:46:42 UTC
There was a new layout suggested to me [1] which I quite like I just haven't come round to implementing it yet.

[1] http://wstaw.org/m/2016/07/22/media-player-t2.png
Comment 4 David Edmundson 2018-02-13 13:04:52 UTC
ok, lets confirm it and someone can make it like that screenshot.
Comment 5 Roman Gilg 2018-03-07 09:13:45 UTC
Git commit d511a766779be67c4343d8cb6b0fcc35c53f6adf by Roman Gilg.
Committed on 07/03/2018 at 09:13.
Pushed by romangilg into branch 'master'.

[mediacontroller applet] Reorder elements

Improve overall look according to bug report and mockup.


Reviewers: #plasma, andreask, broulik

Reviewed By: #plasma, andreask, broulik

Subscribers: ngraham, abetts, nicolasfella, broulik, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D11063

M  +195  -176  applets/mediacontroller/contents/ui/ExpandedRepresentation.qml