From a6f78e7ea70be884742dcb0c53751426976de64c Mon Sep 17 00:00:00 2001 From: Fatih Uzunoglu Date: Wed, 11 Sep 2024 16:07:25 +0300 Subject: [PATCH] qt: reduce playlist delegate vertical padding Although having excessive padding makes it possible to interact with the delegate using touch control easier, it also causes the playlist to show less items at once. VLC is primarily a desktop application, although we should care about touch support, we should not optimize the design for touch. Reducing the vertical padding from 6dp to 2dp seems to be good compromise. It is still very much possible to use touch, but also looks okay for a desktop application. --- modules/gui/qt/playlist/qml/PlaylistDelegate.qml | 2 +- modules/gui/qt/style/VLCStyle.qml | 2 -- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/modules/gui/qt/playlist/qml/PlaylistDelegate.qml b/modules/gui/qt/playlist/qml/PlaylistDelegate.qml index f61b5926d732..33be72772e17 100644 --- a/modules/gui/qt/playlist/qml/PlaylistDelegate.qml +++ b/modules/gui/qt/playlist/qml/PlaylistDelegate.qml @@ -64,7 +64,7 @@ T.Control { hoverEnabled: true - verticalPadding: VLCStyle.playlistDelegate_verticalPadding + verticalPadding: VLCStyle.margin_xxxsmall leftPadding: VLCStyle.margin_normal diff --git a/modules/gui/qt/style/VLCStyle.qml b/modules/gui/qt/style/VLCStyle.qml index 19ce6ee17087..13cbe9afe180 100644 --- a/modules/gui/qt/style/VLCStyle.qml +++ b/modules/gui/qt/style/VLCStyle.qml @@ -232,8 +232,6 @@ QtObject { readonly property int artistBanner_height: MainCtx.dp(200, scale) - readonly property int playlistDelegate_verticalPadding: MainCtx.dp(6, scale) - //global application size, updated by the root widget property int appWidth: 0 property int appHeight: 0