Changeset 108005 in vbox for trunk/src/VBox/Frontends

Timestamp:

Jan 31, 2025 8:45:02 PM (3 months ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

167277

Message:

FE/Qt: bugref:10814: VBox Manager / Tools pane: A bit of cleanup for tool model.

Location:

trunk/src/VBox/Frontends/VirtualBox/src/manager/tools

Files:

• UIToolsModel.cpp (modified) (2 diffs)
• UIToolsModel.h (modified) (1 diff)

trunk/src/VBox/Frontends/VirtualBox/src/manager/tools/UIToolsModel.cpp

@@ -434 +434 @@
                         /* Which item we just clicked? */
                         UIToolsItem *pClickedItem = qgraphicsitem_cast<UIToolsItem*>(pItemUnderMouse);
-                        /* Make clicked item the current one: */
-                        if (pClickedItem && pClickedItem->isEnabled())
+                        if (pClickedItem)
                         {
-                            setCurrentItem(pClickedItem);
-                            /* Close the widget in popup mode only: */
-                            if (tools()->isPopup())
-                                close();
-                            return true;
+                            /* Make clicked item the current one: */
+                            if (pClickedItem->isEnabled())
+                            {
+                                setCurrentItem(pClickedItem);
+                                /* Close the widget in popup mode only: */
+                                if (tools()->isPopup())
+                                    close();
+                                return true;
+                            }
                         }
                         break;
@@ -689 +692 @@
         case ToolsModelData_Margin: return 0;
         case ToolsModelData_Spacing: return 1;
-        case ToolsModelData_MajorSpacing: return 20;
+        case ToolsModelData_MajorSpacing: return 0;
 
         /* Default: */

trunk/src/VBox/Frontends/VirtualBox/src/manager/tools/UIToolsModel.h

@@ -130 +130 @@
         UIToolsItem *item(UIToolType enmType) const;
 
-        /** Returns whether at least one item hovered: */
+        /** Returns whether at least one item hovered. */
         bool isAtLeastOneItemHovered() const;
     /** @} */