VirtualBox

Changeset 108157 in vbox


Ignore:
Timestamp:
Feb 11, 2025 1:36:49 PM (10 days ago)
Author:
vboxsync
svn:sync-xref-src-repo-rev:
167470
Message:

FE/Qt: bugref:10814: VBox Manager / Chooser pane: Cleanup all the stuff related to global item itself.

Location:
trunk/src/VBox/Frontends/VirtualBox
Files:
2 deleted
8 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/VBox/Frontends/VirtualBox/Makefile.kmk

    r108145 r108157  
    598598        src/manager/chooser/UIChooserItem.h \
    599599        src/manager/chooser/UIChooserItemGroup.h \
    600         src/manager/chooser/UIChooserItemGlobal.h \
    601600        src/manager/chooser/UIChooserItemMachine.h \
    602601        src/manager/chooser/UIChooserNode.h \
     
    11231122        src/manager/chooser/UIChooserItem.cpp \
    11241123        src/manager/chooser/UIChooserItemGroup.cpp \
    1125         src/manager/chooser/UIChooserItemGlobal.cpp \
    11261124        src/manager/chooser/UIChooserItemMachine.cpp \
    11271125        src/manager/chooser/UIChooserNode.cpp \
  • trunk/src/VBox/Frontends/VirtualBox/src/manager/chooser/UIChooserHandlerKeyboard.cpp

    r106061 r108157  
    112112                        {
    113113                            pPossiblePreviousItem = model()->navigationItems().at(i);
    114                             if ((      pCurrentItem->type() == UIChooserNodeType_Global
    115                                     && pPossiblePreviousItem->type() == UIChooserNodeType_Global)
    116                                 || (   pCurrentItem->type() != UIChooserNodeType_Global
    117                                     && pPossiblePreviousItem->type() != UIChooserNodeType_Global))
    118                                 pPreviousItem = pPossiblePreviousItem;
     114                            pPreviousItem = pPossiblePreviousItem;
    119115                        }
    120116                    }
     
    217213                        {
    218214                            pPossibleNextItem = model()->navigationItems().at(i);
    219                             if ((      pCurrentItem->type() == UIChooserNodeType_Global
    220                                     && pPossibleNextItem->type() == UIChooserNodeType_Global)
    221                                 || (   pCurrentItem->type() != UIChooserNodeType_Global
    222                                     && pPossibleNextItem->type() != UIChooserNodeType_Global))
    223                                 pNextItem = pPossibleNextItem;
     215                            pNextItem = pPossibleNextItem;
    224216                        }
    225217                    }
  • trunk/src/VBox/Frontends/VirtualBox/src/manager/chooser/UIChooserHandlerMouse.cpp

    r108147 r108157  
    3333#include "UIChooserModel.h"
    3434#include "UIChooserItemGroup.h"
    35 #include "UIChooserItemGlobal.h"
    3635#include "UIChooserItemMachine.h"
    3736
     
    7877                if (UIChooserItemGroup *pGroupItem = qgraphicsitem_cast<UIChooserItemGroup*>(pItemUnderMouse))
    7978                    pClickedItem = pGroupItem;
    80                 /* Or a global one? */
    81                 else if (UIChooserItemGlobal *pGlobalItem = qgraphicsitem_cast<UIChooserItemGlobal*>(pItemUnderMouse))
    82                 {
    83                     const QPoint itemCursorPos = pGlobalItem->mapFromScene(scenePos).toPoint();
    84                     if (   pGlobalItem->isToolButtonArea(itemCursorPos)
    85                         && (   model()->firstSelectedItem() == pGlobalItem
    86                             || pGlobalItem->isHovered()))
    87                     {
    88                         model()->handleToolButtonClick(pGlobalItem);
    89                         if (model()->firstSelectedItem() != pGlobalItem)
    90                             pClickedItem = pGlobalItem;
    91                     }
    92                     else
    93                         pClickedItem = pGlobalItem;
    94                 }
    9579                /* Or a machine one? */
    9680                else if (UIChooserItemMachine *pMachineItem = qgraphicsitem_cast<UIChooserItemMachine*>(pItemUnderMouse))
     
    130114                        QList<UIChooserItem*> filteredItems;
    131115                        foreach (UIChooserItem *pIteratedItem, items)
    132                         {
    133                             /* So, the logic is to add intermediate item if
    134                              * - first and intermediate selected items are global or
    135                              * - first and intermediate selected items are NOT global. */
    136                             if (   (   pFirstItem->type() == UIChooserNodeType_Global
    137                                     && pIteratedItem->type() == UIChooserNodeType_Global)
    138                                 || (   pFirstItem->type() != UIChooserNodeType_Global
    139                                     && pIteratedItem->type() != UIChooserNodeType_Global))
    140                                 filteredItems << pIteratedItem;
    141                         }
     116                            filteredItems << pIteratedItem;
    142117                        /* Make that list selected: */
    143118                        model()->setSelectedItems(filteredItems);
     
    154129                        else
    155130                        {
    156                             /* So, the logic is to add newly clicked item if
    157                              * - previously and newly selected items are global or
    158                              * - previously and newly selected items are NOT global. */
    159131                            UIChooserItem *pFirstItem = model()->firstSelectedItem();
    160132                            AssertPtrReturn(pFirstItem, false); // is failure possible?
    161                             if (   (   pFirstItem->type() == UIChooserNodeType_Global
    162                                     && pClickedItem->type() == UIChooserNodeType_Global)
    163                                 || (   pFirstItem->type() != UIChooserNodeType_Global
    164                                     && pClickedItem->type() != UIChooserNodeType_Global))
    165                                 model()->addToSelectedItems(pClickedItem);
     133                            model()->addToSelectedItems(pClickedItem);
    166134                        }
    167135                        /* Make clicked item current one: */
     
    185153                if (UIChooserItemGroup *pGroupItem = qgraphicsitem_cast<UIChooserItemGroup*>(pItemUnderMouse))
    186154                    pClickedItem = pGroupItem;
    187                 /* Or a global one? */
    188                 else if (UIChooserItemGlobal *pGlobalItem = qgraphicsitem_cast<UIChooserItemGlobal*>(pItemUnderMouse))
    189                     pClickedItem = pGlobalItem;
    190155                /* Or a machine one? */
    191156                else if (UIChooserItemMachine *pMachineItem = qgraphicsitem_cast<UIChooserItemMachine*>(pItemUnderMouse))
  • trunk/src/VBox/Frontends/VirtualBox/src/manager/chooser/UIChooserItem.cpp

    r108147 r108157  
    4242#include "UIChooserItem.h"
    4343#include "UIChooserItemGroup.h"
    44 #include "UIChooserItemGlobal.h"
    4544#include "UIChooserItemMachine.h"
    4645#include "UIChooserView.h"
     
    351350}
    352351
    353 UIChooserItemGlobal *UIChooserItem::toGlobalItem()
    354 {
    355     UIChooserItemGlobal *pItem = qgraphicsitem_cast<UIChooserItemGlobal*>(this);
    356     AssertMsg(pItem, ("Trying to cast invalid item type to UIChooserItemGlobal!"));
    357     return pItem;
    358 }
    359 
    360352UIChooserItemMachine *UIChooserItem::toMachineItem()
    361353{
  • trunk/src/VBox/Frontends/VirtualBox/src/manager/chooser/UIChooserItem.h

    r108147 r108157  
    5151class UIActionPool;
    5252class UIChooserItemGroup;
    53 class UIChooserItemGlobal;
    5453class UIChooserItemMachine;
    5554class UIChooserModel;
     
    116115        /** Casts item to group one. */
    117116        UIChooserItemGroup *toGroupItem();
    118         /** Casts item to global one. */
    119         UIChooserItemGlobal *toGlobalItem();
    120117        /** Casts item to machine one. */
    121118        UIChooserItemMachine *toMachineItem();
  • trunk/src/VBox/Frontends/VirtualBox/src/manager/chooser/UIChooserItemGroup.cpp

    r108147 r108157  
    4040
    4141/* GUI includes: */
    42 #include "UIChooserItemGlobal.h"
    4342#include "UIChooserItemGroup.h"
    4443#include "UIChooserItemMachine.h"
     
    374373    switch (type)
    375374    {
    376         case UIChooserNodeType_Any: return items(UIChooserNodeType_Global) + items(UIChooserNodeType_Group) + items(UIChooserNodeType_Machine);
    377         case UIChooserNodeType_Global: return m_globalItems;
     375        case UIChooserNodeType_Any: return items(UIChooserNodeType_Group) + items(UIChooserNodeType_Machine);
    378376        case UIChooserNodeType_Group: return m_groupItems;
    379377        case UIChooserNodeType_Machine: return m_machineItems;
     
    388386    switch (pItem->type())
    389387    {
    390         case UIChooserNodeType_Global:
    391         {
    392             AssertMsg(!m_globalItems.contains(pItem), ("Global-item already added!"));
    393             if (iPosition < 0 || iPosition >= m_globalItems.size())
    394             {
    395                 m_pLayoutGlobal->addItem(pItem);
    396                 m_globalItems.append(pItem);
    397             }
    398             else
    399             {
    400                 m_pLayoutGlobal->insertItem(iPosition, pItem);
    401                 m_globalItems.insert(iPosition, pItem);
    402             }
    403             break;
    404         }
    405388        case UIChooserNodeType_Group:
    406389        {
     
    452435    switch (pItem->type())
    453436    {
    454         case UIChooserNodeType_Global:
    455         {
    456             AssertMsg(m_globalItems.contains(pItem), ("Global-item was not found!"));
    457             m_globalItems.removeAt(m_globalItems.indexOf(pItem));
    458             m_pLayoutGlobal->removeItem(pItem);
    459             break;
    460         }
    461437        case UIChooserNodeType_Group:
    462438        {
     
    529505    /* Search among all the children, but machines first: */
    530506    foreach (UIChooserItem *pItem, items(UIChooserNodeType_Machine))
    531         if (UIChooserItem *pFoundItem = pItem->searchForItem(strSearchTag, iSearchFlags))
    532             return pFoundItem;
    533     foreach (UIChooserItem *pItem, items(UIChooserNodeType_Global))
    534507        if (UIChooserItem *pFoundItem = pItem->searchForItem(strSearchTag, iSearchFlags))
    535508            return pFoundItem;
     
    11601133    /* Delete all the items: */
    11611134    while (!m_groupItems.isEmpty()) { delete m_groupItems.last(); }
    1162     while (!m_globalItems.isEmpty()) { delete m_globalItems.last(); }
    11631135    while (!m_machineItems.isEmpty()) { delete m_machineItems.last(); }
    11641136
     
    12451217    foreach (UIChooserNode *pNode, pCopyFrom->nodes(UIChooserNodeType_Group))
    12461218        new UIChooserItemGroup(this, pNode->toGroupNode());
    1247     foreach (UIChooserNode *pNode, pCopyFrom->nodes(UIChooserNodeType_Global))
    1248         new UIChooserItemGlobal(this, pNode->toGlobalNode());
    12491219    foreach (UIChooserNode *pNode, pCopyFrom->nodes(UIChooserNodeType_Machine))
    12501220        new UIChooserItemMachine(this, pNode->toMachineNode());
     
    15461516void UIChooserItemGroup::updateLayoutSpacings()
    15471517{
    1548     m_pLayout->setItemSpacing(0, m_globalItems.isEmpty() ? 0 : 1);
    1549     m_pLayout->setItemSpacing(1, m_groupItems.isEmpty() ? 0 : 1);
    1550     m_pLayout->setItemSpacing(2, m_machineItems.isEmpty() ? 0 : 1);
     1518    m_pLayout->setItemSpacing(0, m_groupItems.isEmpty() ? 0 : 1);
     1519    m_pLayout->setItemSpacing(1, m_machineItems.isEmpty() ? 0 : 1);
    15511520}
    15521521
  • trunk/src/VBox/Frontends/VirtualBox/src/manager/chooser/UIChooserItemGroup.h

    r108147 r108157  
    378378        QGraphicsLinearLayout *m_pLayoutMachine;
    379379
    380         /** Holds the global children list. */
    381         QList<UIChooserItem*>  m_globalItems;
    382380        /** Holds the group children list. */
    383381        QList<UIChooserItem*>  m_groupItems;
  • trunk/src/VBox/Frontends/VirtualBox/src/manager/chooser/UIChooserModel.cpp

    r108155 r108157  
    4141#include "UIChooserHandlerKeyboard.h"
    4242#include "UIChooserItemGroup.h"
    43 #include "UIChooserItemGlobal.h"
    4443#include "UIChooserItemMachine.h"
    4544#include "UIChooserModel.h"
     
    376375                pItem = navigationItems().at(idxAfter);
    377376                if (   !selectedItems().contains(pItem)
    378                     && (   pItem->type() == UIChooserNodeType_Machine
    379                         || pItem->type() == UIChooserNodeType_Global))
     377                    && pItem->type() == UIChooserNodeType_Machine)
    380378                    return pItem;
    381379                ++idxAfter;
     
    385383                pItem = navigationItems().at(idxBefore);
    386384                if (   !selectedItems().contains(pItem)
    387                     && (   pItem->type() == UIChooserNodeType_Machine
    388                         || pItem->type() == UIChooserNodeType_Global))
     385                    && pItem->type() == UIChooserNodeType_Machine)
    389386                    return pItem;
    390387                --idxBefore;
     
    415412    if (selectedItemsSet.intersects(matchedItems))
    416413        setSelectedItem(findClosestUnselectedItem());
    417 
    418     /* If global item is currently chosen, selection should be invalidated: */
    419     if (firstSelectedItem() && firstSelectedItem()->type() == UIChooserNodeType_Global)
    420         emit sigSelectionInvalidated();
    421414}
    422415
     
    14161409void UIChooserModel::prepareContextMenu()
    14171410{
    1418     /* Context menu for global(s): */
    1419     m_localMenus[UIChooserNodeType_Global] = new QMenu;
    1420     if (QMenu *pMenuGlobal = m_localMenus.value(UIChooserNodeType_Global))
    1421     {
    1422 #ifdef VBOX_WS_MAC
    1423         pMenuGlobal->addAction(actionPool()->action(UIActionIndex_M_Application_S_About));
    1424         pMenuGlobal->addSeparator();
    1425         pMenuGlobal->addAction(actionPool()->action(UIActionIndex_M_Application_S_Preferences));
    1426         pMenuGlobal->addSeparator();
    1427         pMenuGlobal->addAction(actionPool()->action(UIActionIndexMN_M_File_S_ImportAppliance));
    1428         pMenuGlobal->addAction(actionPool()->action(UIActionIndexMN_M_File_S_ExportAppliance));
    1429 # ifdef VBOX_GUI_WITH_EXTRADATA_MANAGER_UI
    1430         pMenuGlobal->addAction(actionPool()->action(UIActionIndexMN_M_File_S_ShowExtraDataManager));
    1431         pMenuGlobal->addSeparator();
    1432 # endif
    1433         pMenuGlobal->addAction(actionPool()->action(UIActionIndexMN_M_File_M_Tools));
    1434 
    1435 #else /* !VBOX_WS_MAC */
    1436 
    1437         pMenuGlobal->addAction(actionPool()->action(UIActionIndex_M_Application_S_Preferences));
    1438         pMenuGlobal->addSeparator();
    1439         pMenuGlobal->addAction(actionPool()->action(UIActionIndexMN_M_File_S_ImportAppliance));
    1440         pMenuGlobal->addAction(actionPool()->action(UIActionIndexMN_M_File_S_ExportAppliance));
    1441         pMenuGlobal->addSeparator();
    1442 # ifdef VBOX_GUI_WITH_EXTRADATA_MANAGER_UI
    1443         pMenuGlobal->addAction(actionPool()->action(UIActionIndexMN_M_File_S_ShowExtraDataManager));
    1444         pMenuGlobal->addSeparator();
    1445 # endif
    1446         pMenuGlobal->addAction(actionPool()->action(UIActionIndexMN_M_File_M_Tools));
    1447         pMenuGlobal->addSeparator();
    1448 # ifdef VBOX_GUI_WITH_NETWORK_MANAGER
    1449         if (gEDataManager->applicationUpdateEnabled())
    1450             pMenuGlobal->addAction(actionPool()->action(UIActionIndex_M_Application_S_CheckForUpdates));
    1451 # endif
    1452 #endif /* !VBOX_WS_MAC */
    1453     }
    1454 
    14551411    /* Context menu for local group(s): */
    14561412    m_localMenus[UIChooserNodeType_Group] = new QMenu;
     
    16301586                switch (pItem->type())
    16311587                {
    1632                     case UIChooserNodeType_Global:
    1633                     {
    1634                         /* Global context menu for all global item cases: */
    1635                         m_localMenus.value(UIChooserNodeType_Global)->exec(pEvent->screenPos());
    1636                         break;
    1637                     }
    16381588                    case UIChooserNodeType_Group:
    16391589                    {
     
    16841634                switch (pItem->type())
    16851635                {
    1686                     case UIChooserNodeType_Global:
    1687                     {
    1688                         /* Global context menu for all global item cases: */
    1689                         m_localMenus.value(UIChooserNodeType_Global)->exec(pEvent->screenPos());
    1690                         break;
    1691                     }
    16921636                    case UIChooserNodeType_Group:
    16931637                    {
     
    17461690    QList<UIChooserItem*> navigationItems;
    17471691
    1748     /* Iterate over all the global-items: */
    1749     foreach (UIChooserItem *pGlobalItem, pItem->items(UIChooserNodeType_Global))
    1750         navigationItems << pGlobalItem;
    17511692    /* Iterate over all the group-items: */
    17521693    foreach (UIChooserItem *pGroupItem, pItem->items(UIChooserNodeType_Group))
Note: See TracChangeset for help on using the changeset viewer.

© 2024 Oracle Support Privacy / Do Not Sell My Info Terms of Use Trademark Policy Automated Access Etiquette