From 0f6a1206e6615d244472363c3d3d828560d12407 Mon Sep 17 00:00:00 2001
From: Uwe Rathmann <Uwe.Rathmann@tigertal.de>
Date: Tue, 4 Jul 2023 07:49:38 +0200
Subject: [PATCH] missing cell padding added for fluent2

---
 examples/gallery/listbox/ListBoxPage.cpp | 4 ----
 skins/fluent2/QskFluent2Skin.cpp         | 5 +++++
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/examples/gallery/listbox/ListBoxPage.cpp b/examples/gallery/listbox/ListBoxPage.cpp
index e0650d87..4393e32e 100644
--- a/examples/gallery/listbox/ListBoxPage.cpp
+++ b/examples/gallery/listbox/ListBoxPage.cpp
@@ -16,10 +16,6 @@ namespace
             : QskSimpleListBox( parent )
         {
             setMargins( QMarginsF( 15, 10, 10, 10 ) );
-
-            // increasing the padding of each row: usually the job of the skin !
-            setPaddingHint( Cell, QMargins( 10, 20, 10, 20 ) );
-
             populate();
 
             setSelectedRow( 5 );
diff --git a/skins/fluent2/QskFluent2Skin.cpp b/skins/fluent2/QskFluent2Skin.cpp
index e1641198..0cd05f89 100644
--- a/skins/fluent2/QskFluent2Skin.cpp
+++ b/skins/fluent2/QskFluent2Skin.cpp
@@ -34,6 +34,7 @@
 
       - hover state is not implemented
       - Indicator subcontrol might be better than using the border of the selection box
+      - cell padding unclear
 
     - using qskDpToPixels ?
  */
@@ -607,6 +608,10 @@ void Editor::setupListViewMetrics()
     using Q = QskListView;
 
     setBoxBorderMetrics( Q::Cell | Q::Selected, { 3, 0, 0, 0 } );
+#if 1
+    // taken from M3 - what are the actual values, TODO ...
+    setPadding( Q::Cell, { 16, 12, 16, 12 } );
+#endif
 }
 
 void Editor::setupListViewColors(