VirtualBox

Ignore:
Timestamp:
Oct 29, 2018 5:57:17 PM (6 years ago)
Author:
vboxsync
Message:

FE/Qt: bugref:9241: UIActionPool & UIActionPoolSelector: Make sure tool-actions shortcut scopes are limited to their dialogs, that solves shortcut collisions between tools.

Location:
trunk/src/VBox/Frontends/VirtualBox/src
Files:
7 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/VBox/Frontends/VirtualBox/src/cloud/UICloudProfileManager.cpp

    r75054 r75161  
    607607void UICloudProfileManagerWidget::prepareActions()
    608608{
     609    /* First of all, add actions which has smaller shortcut scope: */
     610    addAction(m_pActionPool->action(UIActionIndexST_M_Cloud_S_Add));
     611    addAction(m_pActionPool->action(UIActionIndexST_M_Cloud_S_Import));
     612    addAction(m_pActionPool->action(UIActionIndexST_M_Cloud_S_Remove));
     613    addAction(m_pActionPool->action(UIActionIndexST_M_Cloud_T_Details));
     614    addAction(m_pActionPool->action(UIActionIndexST_M_Cloud_S_Help));
     615
    609616    /* Connect actions: */
    610617    connect(m_pActionPool->action(UIActionIndexST_M_Cloud_S_Add), &QAction::triggered,

  • trunk/src/VBox/Frontends/VirtualBox/src/globals/UIActionPool.cpp

    r75136 r75161  
    12231223        : UIActionToggle(pParent,
    12241224                         ":/log_viewer_find_32px.png", ":/log_viewer_find_disabled_32px.png")
    1225     {}
     1225    {
     1226        setShortcutContext(Qt::WidgetWithChildrenShortcut);
     1227    }
    12261228
    12271229protected:
    … …  
    12601262        : UIActionToggle(pParent,
    12611263                         ":/log_viewer_filter_32px.png", ":/log_viewer_filter_disabled_32px.png")
    1262     {}
     1264    {
     1265        setShortcutContext(Qt::WidgetWithChildrenShortcut);
     1266    }
    12631267
    12641268protected:
    … …  
    12971301        : UIActionToggle(pParent,
    12981302                         ":/log_viewer_bookmark_32px.png", ":/log_viewer_bookmark_disabled_32px.png")
    1299     {}
     1303    {
     1304        setShortcutContext(Qt::WidgetWithChildrenShortcut);
     1305    }
    13001306
    13011307protected:
    … …  
    13341340        : UIActionToggle(pParent,
    13351341                         ":/log_viewer_settings_32px.png", ":/log_viewer_settings_disabled_32px.png")
    1336     {}
     1342    {
     1343        setShortcutContext(Qt::WidgetWithChildrenShortcut);
     1344    }
    13371345
    13381346protected:
    … …  
    13711379        : UIActionSimple(pParent,
    13721380                         ":/log_viewer_refresh_32px.png", ":/log_viewer_refresh_disabled_32px.png")
    1373     {}
     1381    {
     1382        setShortcutContext(Qt::WidgetWithChildrenShortcut);
     1383    }
    13741384
    13751385protected:
    … …  
    14081418        : UIActionSimple(pParent,
    14091419                         ":/log_viewer_save_32px.png", ":/log_viewer_save_disabled_32px.png")
    1410     {}
     1420    {
     1421        setShortcutContext(Qt::WidgetWithChildrenShortcut);
     1422    }
    14111423
    14121424protected:

  • trunk/src/VBox/Frontends/VirtualBox/src/globals/UIActionPoolSelector.cpp

    r75143 r75161  
    15941594                         ":/snapshot_take_32px.png", ":/snapshot_take_16px.png",
    15951595                         ":/snapshot_take_disabled_32px.png", ":/snapshot_take_disabled_16px.png")
    1596     {}
     1596    {
     1597        setShortcutContext(Qt::WidgetWithChildrenShortcut);
     1598    }
    15971599
    15981600protected:
    … …  
    16321634                         ":/snapshot_delete_32px.png", ":/snapshot_delete_16px.png",
    16331635                         ":/snapshot_delete_disabled_32px.png", ":/snapshot_delete_disabled_16px.png")
    1634     {}
     1636    {
     1637        setShortcutContext(Qt::WidgetWithChildrenShortcut);
     1638    }
    16351639
    16361640protected:
    … …  
    16701674                         ":/snapshot_restore_32px.png", ":/snapshot_restore_16px.png",
    16711675                         ":/snapshot_restore_disabled_32px.png", ":/snapshot_restore_disabled_16px.png")
    1672     {}
     1676    {
     1677        setShortcutContext(Qt::WidgetWithChildrenShortcut);
     1678    }
    16731679
    16741680protected:
    … …  
    17071713        : UIActionToggle(pParent)
    17081714    {
     1715        setShortcutContext(Qt::WidgetWithChildrenShortcut);
    17091716        setIcon(UIIconPool::iconSetFull(":/snapshot_show_details_32px.png", ":/snapshot_show_details_16px.png",
    17101717                                        ":/snapshot_show_details_disabled_32px.png", ":/snapshot_show_details_disabled_16px.png"));
    … …  
    17221729    virtual QKeySequence defaultShortcut(UIActionPoolType) const /* override */
    17231730    {
    1724         return QKeySequence("Ctrl+Space");
     1731        return QKeySequence("Ctrl+Shift+P");
    17251732    }
    17261733
    … …  
    17471754                         ":/vm_clone_32px.png", ":/vm_clone_16px.png",
    17481755                         ":/vm_clone_disabled_32px.png", ":/vm_clone_disabled_16px.png")
    1749     {}
     1756    {
     1757        setShortcutContext(Qt::WidgetWithChildrenShortcut);
     1758    }
    17501759
    17511760protected:
    … …  
    18131822        : UIActionSimple(pParent)
    18141823    {
     1824        setShortcutContext(Qt::WidgetWithChildrenShortcut);
    18151825        setIcon(0, UIIconPool::iconSetFull(":/hd_add_32px.png",          ":/hd_add_16px.png",
    18161826                                           ":/hd_add_disabled_32px.png", ":/hd_add_disabled_16px.png"));
    … …  
    18561866        : UIActionSimple(pParent)
    18571867    {
     1868        setShortcutContext(Qt::WidgetWithChildrenShortcut);
    18581869        setIcon(0, UIIconPool::iconSetFull(":/hd_copy_32px.png",          ":/hd_copy_16px.png",
    18591870                                           ":/hd_copy_disabled_32px.png", ":/hd_copy_disabled_16px.png"));
    … …  
    18991910        : UIActionSimple(pParent)
    19001911    {
     1912        setShortcutContext(Qt::WidgetWithChildrenShortcut);
    19011913        setIcon(0, UIIconPool::iconSetFull(":/hd_move_32px.png",          ":/hd_move_16px.png",
    19021914                                           ":/hd_move_disabled_32px.png", ":/hd_move_disabled_16px.png"));
    … …  
    19421954        : UIActionSimple(pParent)
    19431955    {
     1956        setShortcutContext(Qt::WidgetWithChildrenShortcut);
    19441957        setIcon(0, UIIconPool::iconSetFull(":/hd_remove_32px.png",          ":/hd_remove_16px.png",
    19451958                                           ":/hd_remove_disabled_32px.png", ":/hd_remove_disabled_16px.png"));
    … …  
    19851998        : UIActionSimple(pParent)
    19861999    {
     2000        setShortcutContext(Qt::WidgetWithChildrenShortcut);
    19872001        setIcon(0, UIIconPool::iconSetFull(":/hd_release_32px.png",          ":/hd_release_16px.png",
    19882002                                           ":/hd_release_disabled_32px.png", ":/hd_release_disabled_16px.png"));
    … …  
    20282042        : UIActionToggle(pParent)
    20292043    {
     2044        setShortcutContext(Qt::WidgetWithChildrenShortcut);
    20302045        setIcon(0, UIIconPool::iconSetFull(":/hd_modify_32px.png",          ":/hd_modify_16px.png",
    20312046                                           ":/hd_modify_disabled_32px.png", ":/hd_modify_disabled_16px.png"));
    … …  
    20472062    virtual QKeySequence defaultShortcut(UIActionPoolType) const /* override */
    20482063    {
    2049         return QKeySequence("Ctrl+Space");
     2064        return QKeySequence("Ctrl+Shift+P");
    20502065    }
    20512066
    … …  
    20722087                         ":/refresh_32px.png",          ":/refresh_16px.png",
    20732088                         ":/refresh_disabled_32px.png", ":/refresh_disabled_16px.png")
    2074     {}
     2089    {
     2090        setShortcutContext(Qt::WidgetWithChildrenShortcut);
     2091    }
    20752092
    20762093protected:
    … …  
    21392156                         ":/host_iface_add_32px.png",          ":/host_iface_add_16px.png",
    21402157                         ":/host_iface_add_disabled_32px.png", ":/host_iface_add_disabled_16px.png")
    2141     {}
     2158    {
     2159        setShortcutContext(Qt::WidgetWithChildrenShortcut);
     2160    }
    21422161
    21432162protected:
    … …  
    21772196                         ":/host_iface_remove_32px.png",          ":/host_iface_remove_16px.png",
    21782197                         ":/host_iface_remove_disabled_32px.png", ":/host_iface_remove_disabled_16px.png")
    2179     {}
     2198    {
     2199        setShortcutContext(Qt::WidgetWithChildrenShortcut);
     2200    }
    21802201
    21812202protected:
    … …  
    22142235        : UIActionToggle(pParent)
    22152236    {
     2237        setShortcutContext(Qt::WidgetWithChildrenShortcut);
    22162238        setIcon(UIIconPool::iconSetFull(":/host_iface_edit_32px.png",          ":/host_iface_edit_16px.png",
    22172239                                        ":/host_iface_edit_disabled_32px.png", ":/host_iface_edit_disabled_16px.png"));
    … …  
    22292251    virtual QKeySequence defaultShortcut(UIActionPoolType) const /* override */
    22302252    {
    2231         return QKeySequence("Ctrl+Space");
     2253        return QKeySequence("Ctrl+Shift+P");
    22322254    }
    22332255
    … …  
    22542276                         ":/refresh_32px.png",          ":/refresh_16px.png",
    22552277                         ":/refresh_disabled_32px.png", ":/refresh_disabled_16px.png")
    2256     {}
     2278    {
     2279        setShortcutContext(Qt::WidgetWithChildrenShortcut);
     2280    }
    22572281
    22582282protected:
    … …  
    23212345                         ":/cloud_profile_add_32px.png",          ":/cloud_profile_add_16px.png",
    23222346                         ":/cloud_profile_add_disabled_32px.png", ":/cloud_profile_add_disabled_16px.png")
    2323     {}
     2347    {
     2348        setShortcutContext(Qt::WidgetWithChildrenShortcut);
     2349    }
    23242350
    23252351protected:
    … …  
    23602386                         ":/cloud_profile_restore_32px.png",          ":/cloud_profile_restore_16px.png",
    23612387                         ":/cloud_profile_restore_disabled_32px.png", ":/cloud_profile_restore_disabled_16px.png")
    2362     {}
     2388    {
     2389        setShortcutContext(Qt::WidgetWithChildrenShortcut);
     2390    }
    23632391
    23642392protected:
    … …  
    23992427                         ":/cloud_profile_remove_32px.png",          ":/cloud_profile_remove_16px.png",
    24002428                         ":/cloud_profile_remove_disabled_32px.png", ":/cloud_profile_remove_disabled_16px.png")
    2401     {}
     2429    {
     2430        setShortcutContext(Qt::WidgetWithChildrenShortcut);
     2431    }
    24022432
    24032433protected:
    … …  
    24372467        : UIActionToggle(pParent)
    24382468    {
     2469        setShortcutContext(Qt::WidgetWithChildrenShortcut);
    24392470        setIcon(UIIconPool::iconSetFull(":/cloud_profile_edit_32px.png",          ":/cloud_profile_edit_16px.png",
    24402471                                        ":/cloud_profile_edit_disabled_32px.png", ":/cloud_profile_edit_disabled_16px.png"));
    … …  
    24522483    virtual QKeySequence defaultShortcut(UIActionPoolType) const /* override */
    24532484    {
    2454         return QKeySequence("Ctrl+Space");
     2485        return QKeySequence("Ctrl+Shift+P");
    24552486    }
    24562487
    … …  
    24782509                         ":/cloud_profile_help_32px.png",          ":/cloud_profile_help_16px.png",
    24792510                         ":/cloud_profile_help_disabled_32px.png", ":/cloud_profile_help_disabled_16px.png")
    2480     {}
     2511    {
     2512        setShortcutContext(Qt::WidgetWithChildrenShortcut);
     2513    }
    24812514
    24822515protected:

  • trunk/src/VBox/Frontends/VirtualBox/src/hostnetwork/UIHostNetworkManager.cpp

    r75027 r75161  
    693693void UIHostNetworkManagerWidget::prepareActions()
    694694{
     695    /* First of all, add actions which has smaller shortcut scope: */
     696    addAction(m_pActionPool->action(UIActionIndexST_M_Network_S_Create));
     697    addAction(m_pActionPool->action(UIActionIndexST_M_Network_S_Remove));
     698    addAction(m_pActionPool->action(UIActionIndexST_M_Network_T_Details));
     699    addAction(m_pActionPool->action(UIActionIndexST_M_Network_S_Refresh));
     700
    695701    /* Connect actions: */
    696702    connect(m_pActionPool->action(UIActionIndexST_M_Network_S_Create), &QAction::triggered,

  • trunk/src/VBox/Frontends/VirtualBox/src/logviewer/UIVMLogViewerWidget.cpp

    r75027 r75161  
    436436void UIVMLogViewerWidget::prepareActions()
    437437{
     438    /* First of all, add actions which has smaller shortcut scope: */
     439    addAction(m_pActionPool->action(UIActionIndex_M_Log_T_Find));
     440    addAction(m_pActionPool->action(UIActionIndex_M_Log_T_Filter));
     441    addAction(m_pActionPool->action(UIActionIndex_M_Log_T_Bookmark));
     442    addAction(m_pActionPool->action(UIActionIndex_M_Log_T_Settings));
     443    addAction(m_pActionPool->action(UIActionIndex_M_Log_S_Refresh));
     444    addAction(m_pActionPool->action(UIActionIndex_M_Log_S_Save));
     445
    438446    /* Connect actions: */
    439447    connect(m_pActionPool->action(UIActionIndex_M_Log_T_Find), &QAction::toggled,

  • trunk/src/VBox/Frontends/VirtualBox/src/medium/UIMediumManager.cpp

    r75027 r75161  
    720720void UIMediumManagerWidget::prepareActions()
    721721{
     722    /* First of all, add actions which has smaller shortcut scope: */
     723    addAction(m_pActionPool->action(UIActionIndexST_M_Medium_S_Add));
     724    addAction(m_pActionPool->action(UIActionIndexST_M_Medium_S_Copy));
     725    addAction(m_pActionPool->action(UIActionIndexST_M_Medium_S_Move));
     726    addAction(m_pActionPool->action(UIActionIndexST_M_Medium_S_Remove));
     727    addAction(m_pActionPool->action(UIActionIndexST_M_Medium_S_Release));
     728    addAction(m_pActionPool->action(UIActionIndexST_M_Medium_T_Details));
     729    addAction(m_pActionPool->action(UIActionIndexST_M_Medium_S_Refresh));
     730
    722731    /* Connect actions: */
    723732    connect(m_pActionPool->action(UIActionIndexST_M_Medium_S_Add), &QAction::triggered,

  • trunk/src/VBox/Frontends/VirtualBox/src/snapshots/UISnapshotPane.cpp

    r75093 r75161  
    12241224void UISnapshotPane::prepareActions()
    12251225{
     1226    /* First of all, add actions which has smaller shortcut scope: */
     1227    addAction(m_pActionPool->action(UIActionIndexST_M_Snapshot_S_Take));
     1228    addAction(m_pActionPool->action(UIActionIndexST_M_Snapshot_S_Delete));
     1229    addAction(m_pActionPool->action(UIActionIndexST_M_Snapshot_S_Restore));
     1230    addAction(m_pActionPool->action(UIActionIndexST_M_Snapshot_T_Properties));
     1231    addAction(m_pActionPool->action(UIActionIndexST_M_Snapshot_S_Clone));
     1232
    12261233    /* Connect actions: */
    12271234    connect(m_pActionPool->action(UIActionIndexST_M_Snapshot_S_Take), &UIAction::triggered,
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