fix: prevent background type from overriding custom item spacing

1. Changed DStyledItemDelegate to initialize itemSpacing with -1 as
default value
2. Added spacing() method to handle negative spacing values
3. Modified setBackgroundType to only set spacing when no custom spacing
was set (spacing < 0)
4. Updated all spacing references to use the new spacing() method

Log: Fixed issue where setting background type would override custom
item spacing values

Influence:
1. Test setting custom item spacing values before/after changing
background types
2. Verify spacing behavior with different background types
(RoundedBackground, ClipCornerBackground)
3. Check layout calculations with custom spacing values
4. Test list view rendering in both LeftToRight and TopToBottom flows

fix: 修复设置背景类型会覆盖自定义间距值的问题

1. 修改 DStyledItemDelegate 默认将 itemSpacing 初始化为 -1
2. 新增 spacing() 方法处理负间距值
3. 修改 setBackgroundType 仅在未设置自定义间距时（spacing < 0）才设置
间距
4. 更新所有间距引用使用新的 spacing() 方法

Log: 修复了设置背景类型会覆盖自定义间距值的问题

Influence:
1. 测试在更改背景类型前后设置自定义间距值
2. 验证不同背景类型(RoundedBackground, ClipCornerBackground)下的间距行为
3. 检查使用自定义间距值时的布局计算
4. 测试列表视图在 LeftToRight 和 TopToBottom 两种流式布局下的渲染效果

Author: 18202781743

src/widgets/dlistview.cpp

@@ -777,12 +777,12 @@ void DListView::setBackgroundType(DStyledItemDelegate::BackgroundType background
     if (DStyledItemDelegate *d = qobject_cast<DStyledItemDelegate *>(itemDelegate())) {
         d->setBackgroundType(backgroundType);
 
-        if (d->backgroundType() == DStyledItemDelegate::RoundedBackground) {
-            d->setItemSpacing(10);
-        } else if (d->backgroundType() == DStyledItemDelegate::ClipCornerBackground) {
-            d->setItemSpacing(1);
-        } else {
-            d->setItemSpacing(0);
+        if (d->spacing() < 0) {
+            if (d->backgroundType() == DStyledItemDelegate::RoundedBackground) {
+                d->setItemSpacing(10);
+            } else if (d->backgroundType() == DStyledItemDelegate::ClipCornerBackground) {
+                d->setItemSpacing(1);
+            }
         }
     }
 }

src/widgets/dstyleditemdelegate.cpp

@@ -516,10 +516,17 @@ class DStyledItemDelegatePrivate : public DCORE_NAMESPACE::DObjectPrivate
         }
     }
 
+    inline int spacing() const
+    {
+        if (itemSpacing < 0) 
+            return 0;
+        return itemSpacing;
+    }
+
     DStyledItemDelegate::BackgroundType backgroundType = DStyledItemDelegate::NoBackground;
     QMargins margins;
     QSize itemSize;
-    int itemSpacing = 0;
+    int itemSpacing = -1;
     QMap<QModelIndex, QList<QPair<QAction*, QRect>>> clickableActionMap;
     QAction *pressedAction = nullptr;
     QList<QPointer<QWidget>> lastWidgets;
@@ -1176,9 +1183,9 @@ QSize DStyledItemDelegate::sizeHint(const QStyleOptionViewItem &option, const QM
     const QListView * lv = qobject_cast<const QListView*>(option.widget);
     if (lv) {
         if (lv->flow() == QListView::LeftToRight) {
-            size.rwidth() += d->itemSpacing;
+            size.rwidth() += d->spacing();
         } else {
-            size.rheight() += d->itemSpacing;
+            size.rheight() += d->spacing();
         }
     }
 
@@ -1353,9 +1360,9 @@ void DStyledItemDelegate::initStyleOption(QStyleOptionViewItem *option, const QM
     const QListView * lv = qobject_cast<const QListView*>(option->widget);
     if (lv) {
         if (lv->flow() == QListView::LeftToRight) {
-            option->rect.adjust(0, 0, 0 - d->itemSpacing, 0);
+            option->rect.adjust(0, 0, 0 - d->spacing(), 0);
         } else {
-            option->rect.adjust(0, 0, 0, 0 - d->itemSpacing);
+            option->rect.adjust(0, 0, 0, 0 - d->spacing());
         }
         if (lv->window() && lv->window()->isActiveWindow()) {
             option->state |= QStyle::State_Active;