Changeset 74085 in vbox for trunk/src/VBox/Frontends/VirtualBox/src/manager/chooser/UIChooserItemGroup.cpp

Timestamp:

Sep 5, 2018 1:30:48 PM (6 years ago)

Author:

vboxsync

Message:

FE/Qt: VirtualBox Manager UI: Chooser pane re-styling: Apply background instead of painting it manually each time.

File:

• trunk/src/VBox/Frontends/VirtualBox/src/manager/chooser/UIChooserItemGroup.cpp (modified) (3 diffs)

trunk/src/VBox/Frontends/VirtualBox/src/manager/chooser/UIChooserItemGroup.cpp

@@ -1630 +1630 @@
 
     /* Prepare color: */
-    QPalette pal = palette();
-    QColor headerColor = pal.color(QPalette::Active,
-                                   model()->currentItems().contains(this) ?
-                                   QPalette::Highlight : QPalette::Midlight);
-    QColor bodyColor = pal.color(QPalette::Active, QPalette::Midlight).darker(headerDarkness());
+    const QPalette pal = palette();
+    const QColor headerColor = pal.color(QPalette::Active,
+                                         model()->currentItems().contains(this) ?
+                                         QPalette::Highlight : QPalette::Midlight);
 
     /* Root-item: */
     if (isRoot())
     {
-        /* Main root-item: */
-        if (isMainRoot())
-        {
-            /* Simple and clear: */
-            pPainter->fillRect(rect, bodyColor);
-        }
         /* Non-main root-item: */
-        else
+        if (!isMainRoot())
         {
             /* Prepare variables: */
-            int iMargin = data(GroupItemData_VerticalMargin).toInt();
-            int iFullHeaderHeight = 2 * iMargin + m_minimumHeaderSize.height();
-            int iFullBodyHeight = rect.height() - iFullHeaderHeight;
+            const int iMargin = data(GroupItemData_VerticalMargin).toInt();
+            const int iFullHeaderHeight = 2 * iMargin + m_minimumHeaderSize.height();
             QRect headerRect = QRect(0, 0, rect.width(), iFullHeaderHeight);
-            QRect bodyRect = QRect(0, iFullHeaderHeight, rect.width(), iFullBodyHeight);
 
             /* Fill background: */
@@ -1660 +1651 @@
             headerGradient.setColorAt(0, headerColor.darker(animatedValue()));
             pPainter->fillRect(headerRect, headerGradient);
-            pPainter->fillRect(bodyRect, bodyColor);
         }
     }
@@ -1667 +1657 @@
     {
         /* Prepare variables: */
-        int iMargin = data(GroupItemData_VerticalMargin).toInt();
-        int iFullHeaderHeight = 2 * iMargin + m_minimumHeaderSize.height();
+        const int iMargin = data(GroupItemData_VerticalMargin).toInt();
+        const int iFullHeaderHeight = 2 * iMargin + m_minimumHeaderSize.height();
 
         /* Calculate top rectangle: */