VirtualBox

Changeset 52567 in vbox for trunk/src


Ignore:
Timestamp:
Sep 2, 2014 1:52:35 PM (11 years ago)
Author:
vboxsync
svn:sync-xref-src-repo-rev:
95835
Message:

FE/Qt: Runtime UI: Menu-bar separators code cleanup/overhaul.

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

Legend:

Unmodified
Added
Removed

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

    r52558 r52567  
    10081008
    10091009
    1010     /* Separator #1? */
    1011     bool fSeparator1 = false;
     1010    /* Separator? */
     1011    bool fSeparator = false;
     1012
    10121013
    10131014    /* 'Contents' action: */
    … …  
    10191020        connect(action(UIActionIndex_Simple_Contents), SIGNAL(triggered()),
    10201021                &msgCenter(), SLOT(sltShowHelpHelpDialog()), Qt::UniqueConnection);
    1021         fSeparator1 = true;
     1022        fSeparator = true;
    10221023    }
    10231024
    … …  
    10301031        connect(action(UIActionIndex_Simple_WebSite), SIGNAL(triggered()),
    10311032                &msgCenter(), SLOT(sltShowHelpWebDialog()), Qt::UniqueConnection);
    1032         fSeparator1 = true;
    1033     }
    1034 
    1035     /* Separator #1: */
    1036     if (fSeparator1)
     1033        fSeparator = true;
     1034    }
     1035
     1036
     1037    /* Separator? */
     1038    if (fSeparator)
     1039    {
    10371040        pMenu->addSeparator();
    1038 
    1039 
    1040     /* Separator #2? */
    1041     bool fSeparator2 = false;
     1041        fSeparator = false;
     1042    }
     1043
    10421044
    10431045    /* 'Reset Warnings' action: */
    … …  
    10491051        connect(action(UIActionIndex_Simple_ResetWarnings), SIGNAL(triggered()),
    10501052                &msgCenter(), SLOT(sltResetSuppressedMessages()), Qt::UniqueConnection);
    1051         fSeparator2 = true;
    1052     }
    1053 
    1054     /* Separator #2: */
    1055     if (fSeparator2)
     1053        fSeparator = true;
     1054    }
     1055
     1056
     1057    /* Separator? */
     1058    if (fSeparator)
     1059    {
    10561060        pMenu->addSeparator();
     1061        fSeparator = false;
     1062    }
    10571063
    10581064
    10591065#ifdef VBOX_GUI_WITH_NETWORK_MANAGER
    1060 # ifndef Q_WS_MAC
    1061     /* Separator #3? */
    1062     bool fSeparator3 = false;
    1063 # endif /* !Q_WS_MAC */
    1064 
    10651066    /* 'Network Manager' action: */
    10661067    const bool fAllowToShowActionNetworkManager = isAllowedInMenuHelp(UIExtraDataMetaDefs::MenuHelpActionType_NetworkAccessManager);
    … …  
    10711072        connect(action(UIActionIndex_Simple_NetworkAccessManager), SIGNAL(triggered()),
    10721073                gNetworkManager, SLOT(show()), Qt::UniqueConnection);
    1073 # ifndef Q_WS_MAC
    1074         fSeparator3 = true;
    1075 # endif /* !Q_WS_MAC */
     1074        fSeparator = true;
    10761075    }
    10771076
    … …  
    10871086            connect(action(UIActionIndex_Simple_CheckForUpdates), SIGNAL(triggered()),
    10881087                    gUpdateManager, SLOT(sltForceCheck()), Qt::UniqueConnection);
    1089 # ifndef Q_WS_MAC
    1090             fSeparator3 = true;
    1091 # endif /* !Q_WS_MAC */
     1088            fSeparator = true;
    10921089        }
    10931090    }
    10941091
    1095 # ifndef Q_WS_MAC
    1096     /* Separator #3: */
    1097     if (fSeparator3)
     1092
     1093    /* Separator? */
     1094    if (fSeparator)
     1095    {
    10981096        pMenu->addSeparator();
    1099 # endif /* !Q_WS_MAC */
     1097        fSeparator = false;
     1098    }
    11001099#endif /* VBOX_GUI_WITH_NETWORK_MANAGER */
    11011100

  • trunk/src/VBox/Frontends/VirtualBox/src/runtime/UIActionPoolRuntime.cpp

    r52558 r52567  
    20262026
    20272027
    2028     /* Separator #1? */
    2029     bool fSeparator1 = false;
     2028    /* Separator: */
     2029    bool fSeparator = false;
     2030
    20302031
    20312032    /* 'Settings Dialog' action: */
    … …  
    20352036    {
    20362037        pMenu->addAction(action(UIActionIndexRT_M_Machine_S_Settings));
    2037         fSeparator1 = true;
     2038        fSeparator = true;
    20382039    }
    20392040
    … …  
    20442045    {
    20452046        pMenu->addAction(action(UIActionIndexRT_M_Machine_S_TakeSnapshot));
    2046         fSeparator1 = true;
     2047        fSeparator = true;
    20472048    }
    20482049
    … …  
    20532054    {
    20542055        pMenu->addAction(action(UIActionIndexRT_M_Machine_S_TakeScreenshot));
    2055         fSeparator1 = true;
     2056        fSeparator = true;
    20562057    }
    20572058
    … …  
    20622063    {
    20632064        pMenu->addAction(action(UIActionIndexRT_M_Machine_S_ShowInformation));
    2064         fSeparator1 = true;
    2065     }
    2066 
    2067     /* Separator #1: */
    2068     if (fSeparator1)
     2065        fSeparator = true;
     2066    }
     2067
     2068
     2069    /* Separator: */
     2070    if (fSeparator)
     2071    {
    20692072        pMenu->addSeparator();
    2070 
    2071 
    2072     /* Separator #2? */
    2073     bool fSeparator2 = false;
     2073        fSeparator = false;
     2074    }
     2075
    20742076
    20752077    /* 'Keyboard' submenu: */
    … …  
    20792081    {
    20802082//        pMenu->addAction(action(UIActionIndexRT_M_Machine_M_Keyboard));
    2081 //        fSeparator2 = true;
     2083//        fSeparator = true;
    20822084    }
    20832085    updateMenuMachineKeyboard();
    … …  
    20892091    {
    20902092        pMenu->addAction(action(UIActionIndexRT_M_Machine_M_Keyboard_S_Settings));
    2091         fSeparator2 = true;
     2093        fSeparator = true;
    20922094    }
    20932095
    … …  
    20982100    {
    20992101//        pMenu->addAction(action(UIActionIndexRT_M_Machine_M_Mouse));
    2100 //        fSeparator2 = true;
     2102//        fSeparator = true;
    21012103    }
    21022104    updateMenuMachineMouse();
    … …  
    21082110    {
    21092111        pMenu->addAction(action(UIActionIndexRT_M_Machine_M_Mouse_T_Integration));
    2110         fSeparator2 = true;
    2111     }
    2112 
    2113     /* Separator #2: */
    2114     if (fSeparator2)
     2112        fSeparator = true;
     2113    }
     2114
     2115
     2116    /* Separator: */
     2117    if (fSeparator)
     2118    {
    21152119        pMenu->addSeparator();
    2116 
    2117 
    2118     /* Separator #3? */
    2119     bool fSeparator3 = false;
     2120        fSeparator = false;
     2121    }
     2122
    21202123
    21212124    /* 'Type CAD' action: */
    … …  
    21252128    {
    21262129        pMenu->addAction(action(UIActionIndexRT_M_Machine_S_TypeCAD));
    2127         fSeparator3 = true;
     2130        fSeparator = true;
    21282131    }
    21292132
    … …  
    21352138    {
    21362139        pMenu->addAction(action(UIActionIndexRT_M_Machine_S_TypeCABS));
    2137         fSeparator3 = true;
     2140        fSeparator = true;
    21382141    }
    21392142#endif /* Q_WS_X11 */
    21402143
    2141     /* Separator #3: */
    2142     if (fSeparator3)
     2144
     2145    /* Separator: */
     2146    if (fSeparator)
     2147    {
    21432148        pMenu->addSeparator();
    2144 
    2145 
    2146     /* Separator #4? */
    2147     bool fSeparator4 = false;
     2149        fSeparator = false;
     2150    }
     2151
    21482152
    21492153    /* 'Pause' action: */
    … …  
    21532157    {
    21542158        pMenu->addAction(action(UIActionIndexRT_M_Machine_T_Pause));
    2155         fSeparator4 = true;
     2159        fSeparator = true;
    21562160    }
    21572161
    … …  
    21622166    {
    21632167        pMenu->addAction(action(UIActionIndexRT_M_Machine_S_Reset));
    2164         fSeparator4 = true;
     2168        fSeparator = true;
    21652169    }
    21662170
    … …  
    21712175    {
    21722176        pMenu->addAction(action(UIActionIndexRT_M_Machine_S_Save));
    2173         fSeparator4 = true;
     2177        fSeparator = true;
    21742178    }
    21752179
    … …  
    21802184    {
    21812185        pMenu->addAction(action(UIActionIndexRT_M_Machine_S_Shutdown));
    2182         fSeparator4 = true;
     2186        fSeparator = true;
    21832187    }
    21842188
    … …  
    21892193    {
    21902194        pMenu->addAction(action(UIActionIndexRT_M_Machine_S_PowerOff));
    2191         fSeparator4 = true;
    2192     }
     2195        fSeparator = true;
     2196    }
     2197
    21932198
    21942199#ifndef Q_WS_MAC
    2195     /* Separator #4: */
    2196     if (fSeparator4)
     2200    /* Separator: */
     2201    if (fSeparator)
     2202    {
    21972203        pMenu->addSeparator();
    2198 #endif /* !Q_WS_MAC */
    2199 
    2200 
    2201 #ifndef Q_WS_MAC
     2204        fSeparator = false;
     2205    }
     2206
     2207
    22022208    /* Close action: */
    22032209    const bool fAllowToShowActionClose = isAllowedInMenuMachine(UIExtraDataMetaDefs::RuntimeMenuMachineActionType_Close);
    … …  
    22562262
    22572263
    2258     /* Visual representation mode flags: */
     2264    /* Separator: */
     2265    bool fSeparator = false;
     2266
     2267
     2268    /* 'Fullscreen' action: */
    22592269    const bool fIsAllowToShowActionFullscreen = isAllowedInMenuView(UIExtraDataMetaDefs::RuntimeMenuViewActionType_Fullscreen);
    2260     const bool fIsAllowToShowActionSeamless   = isAllowedInMenuView(UIExtraDataMetaDefs::RuntimeMenuViewActionType_Seamless);
    2261     const bool fIsAllowToShowActionScale      = isAllowedInMenuView(UIExtraDataMetaDefs::RuntimeMenuViewActionType_Scale);
    2262 
    2263     /* 'Fullscreen' action: */
    22642270    action(UIActionIndexRT_M_View_T_Fullscreen)->setEnabled(fIsAllowToShowActionFullscreen);
    22652271    if (fIsAllowToShowActionFullscreen)
     2272    {
    22662273        pMenu->addAction(action(UIActionIndexRT_M_View_T_Fullscreen));
     2274        fSeparator = true;
     2275    }
    22672276
    22682277    /* 'Seamless' action: */
     2278    const bool fIsAllowToShowActionSeamless   = isAllowedInMenuView(UIExtraDataMetaDefs::RuntimeMenuViewActionType_Seamless);
    22692279    action(UIActionIndexRT_M_View_T_Seamless)->setEnabled(fIsAllowToShowActionSeamless);
    22702280    if (fIsAllowToShowActionSeamless)
     2281    {
    22712282        pMenu->addAction(action(UIActionIndexRT_M_View_T_Seamless));
     2283        fSeparator = true;
     2284    }
    22722285
    22732286    /* 'Scale' action: */
     2287    const bool fIsAllowToShowActionScale      = isAllowedInMenuView(UIExtraDataMetaDefs::RuntimeMenuViewActionType_Scale);
    22742288    action(UIActionIndexRT_M_View_T_Scale)->setEnabled(fIsAllowToShowActionScale);
    22752289    if (fIsAllowToShowActionScale)
     2290    {
    22762291        pMenu->addAction(action(UIActionIndexRT_M_View_T_Scale));
    2277 
    2278     /* Visual representation mode separator: */
    2279     if (fIsAllowToShowActionFullscreen || fIsAllowToShowActionSeamless || fIsAllowToShowActionScale)
     2292        fSeparator = true;
     2293    }
     2294
     2295
     2296    /* Separator: */
     2297    if (fSeparator)
     2298    {
    22802299        pMenu->addSeparator();
    2281 
    2282 
    2283     /* Separator #1? */
    2284     bool fSeparator1 = false;
     2300        fSeparator = false;
     2301    }
     2302
    22852303
    22862304    /* 'Adjust Window' action: */
    … …  
    22902308    {
    22912309        pMenu->addAction(action(UIActionIndexRT_M_View_S_AdjustWindow));
    2292         fSeparator1 = true;
     2310        fSeparator = true;
    22932311    }
    22942312
    … …  
    22992317    {
    23002318        pMenu->addAction(action(UIActionIndexRT_M_View_T_GuestAutoresize));
    2301         fSeparator1 = true;
    2302     }
    2303 
    2304     /* Separator #1: */
    2305     if (fSeparator1)
     2319        fSeparator = true;
     2320    }
     2321
     2322
     2323    /* Separator: */
     2324    if (fSeparator)
     2325    {
    23062326        pMenu->addSeparator();
    2307 
    2308 
    2309     /* Separator #2? */
    2310     bool fSeparator2 = false;
     2327        fSeparator = false;
     2328    }
     2329
    23112330
    23122331    /* 'Menu Bar' submenu: */
    … …  
    23162335    {
    23172336        pMenu->addAction(action(UIActionIndexRT_M_View_M_MenuBar));
    2318         fSeparator2 = true;
     2337        fSeparator = true;
    23192338    }
    23202339    updateMenuViewMenuBar();
    … …  
    23262345    {
    23272346        pMenu->addAction(action(UIActionIndexRT_M_View_M_StatusBar));
    2328         fSeparator2 = true;
     2347        fSeparator = true;
    23292348    }
    23302349    updateMenuViewStatusBar();
    23312350
    2332     /* Separator #2: */
    2333     if (fSeparator2)
     2351
     2352    /* Separator: */
     2353    if (fSeparator)
     2354    {
    23342355        pMenu->addSeparator();
     2356        fSeparator = false;
     2357    }
    23352358
    23362359
    … …  
    23792402
    23802403
    2381     /* Separator #1? */
    2382     bool fSeparator1 = false;
     2404    /* Separator: */
     2405    bool fSeparator = false;
     2406
    23832407
    23842408    /* 'Adjust Window' action: */
    … …  
    23882412    {
    23892413        pMenu->addAction(action(UIActionIndexRT_M_View_S_AdjustWindow));
    2390         fSeparator1 = true;
     2414        fSeparator = true;
    23912415    }
    23922416
    … …  
    23972421    {
    23982422        pMenu->addAction(action(UIActionIndexRT_M_View_T_GuestAutoresize));
    2399         fSeparator1 = true;
    2400     }
    2401 
    2402     /* Separator #1: */
    2403     if (fSeparator1)
     2423        fSeparator = true;
     2424    }
     2425
     2426
     2427    /* Separator: */
     2428    if (fSeparator)
     2429    {
    24042430        pMenu->addSeparator();
     2431        fSeparator = false;
     2432    }
    24052433
    24062434
    … …  
    25942622
    25952623
    2596     /* Separator #1? */
    2597     bool fSeparator1 = false;
     2624    /* Separator: */
     2625    bool fSeparator = false;
     2626
    25982627
    25992628    /* 'Hard Drives' submenu: */
    … …  
    26032632    {
    26042633        pMenu->addAction(action(UIActionIndexRT_M_Devices_M_HardDrives));
    2605         fSeparator1 = true;
     2634        fSeparator = true;
    26062635    }
    26072636    updateMenuDevicesHardDrives();
    … …  
    26132642    {
    26142643        pMenu->addAction(action(UIActionIndexRT_M_Devices_M_OpticalDevices));
    2615         fSeparator1 = true;
     2644        fSeparator = true;
    26162645    }
    26172646
    … …  
    26222651    {
    26232652        pMenu->addAction(action(UIActionIndexRT_M_Devices_M_FloppyDevices));
    2624         fSeparator1 = true;
     2653        fSeparator = true;
    26252654    }
    26262655
    … …  
    26312660    {
    26322661        pMenu->addAction(action(UIActionIndexRT_M_Devices_M_Network));
    2633         fSeparator1 = true;
     2662        fSeparator = true;
    26342663    }
    26352664    updateMenuDevicesNetwork();
    … …  
    26412670    {
    26422671        pMenu->addAction(action(UIActionIndexRT_M_Devices_M_USBDevices));
    2643         fSeparator1 = true;
     2672        fSeparator = true;
    26442673    }
    26452674    updateMenuDevicesUSBDevices();
    … …  
    26512680    {
    26522681        pMenu->addAction(action(UIActionIndexRT_M_Devices_M_WebCams));
    2653         fSeparator1 = true;
     2682        fSeparator = true;
    26542683    }
    26552684
    … …  
    26602689    {
    26612690        pMenu->addAction(action(UIActionIndexRT_M_Devices_M_SharedClipboard));
    2662         fSeparator1 = true;
     2691        fSeparator = true;
    26632692    }
    26642693
    … …  
    26692698    {
    26702699        pMenu->addAction(action(UIActionIndexRT_M_Devices_M_DragAndDrop));
    2671         fSeparator1 = true;
     2700        fSeparator = true;
    26722701    }
    26732702
    … …  
    26782707    {
    26792708        pMenu->addAction(action(UIActionIndexRT_M_Devices_M_SharedFolders));
    2680         fSeparator1 = true;
     2709        fSeparator = true;
    26812710    }
    26822711    updateMenuDevicesSharedFolders();
    26832712
    2684     /* Separator #1: */
    2685     if (fSeparator1)
     2713
     2714    /* Separator: */
     2715    if (fSeparator)
     2716    {
    26862717        pMenu->addSeparator();
    2687 
    2688 
    2689     /* Separator #2? */
    2690     bool fSeparator2 = false;
     2718        fSeparator = false;
     2719    }
     2720
    26912721
    26922722    /* 'VRDE Server' action: */
    … …  
    26962726    {
    26972727        pMenu->addAction(action(UIActionIndexRT_M_Devices_T_VRDEServer));
    2698         fSeparator2 = true;
     2728        fSeparator = true;
    26992729    }
    27002730
    … …  
    27052735    {
    27062736        pMenu->addAction(action(UIActionIndexRT_M_Devices_M_VideoCapture_T_Start));
    2707         fSeparator2 = true;
     2737        fSeparator = true;
    27082738    }
    27092739    updateMenuDevicesVideoCapture();
    27102740
    2711     /* Separator #2: */
    2712     if (fSeparator2)
     2741
     2742    /* Separator: */
     2743    if (fSeparator)
     2744    {
    27132745        pMenu->addSeparator();
     2746        fSeparator = false;
     2747    }
    27142748
    27152749
Note: See TracChangeset for help on using the changeset viewer.

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