Changeset 13869 in vbox

Timestamp:

Nov 5, 2008 2:38:47 PM (16 years ago)

Author:

vboxsync

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

38867

Message:

FE/Qt: VM Selector: Don't set the Refresh action shortcut on the Refresh button in the details view since it's redundant.

Location:

trunk/src/VBox/Frontends

Files:

• VirtualBox/src/VBoxSelectorWnd.cpp (modified) (1 diff)
• VirtualBox4/src/VBoxSelectorWnd.cpp (modified) (2 diffs)

trunk/src/VBox/Frontends/VirtualBox/src/VBoxSelectorWnd.cpp

@@ -199 +199 @@
     {
         mRefreshButton->setText (mRefreshAction->menuText());
-        /* If we set the combination of the text label and icon
-         * below, we lose the Alt+R shortcut functionality... */
+        /* By setting set the combination of the text label and icon below, we
+         * lose the Alt+R shortcut functionality, so no need to display an
+         * underscore contained in menuText(), therefore use just text() */
         mRefreshButton->setTextLabel (mRefreshAction->text());
-        mRefreshButton->setAccel (mRefreshAction->accel());
         mRefreshButton->setIconSet (mRefreshAction->iconSet());
         mRefreshButton->setTextPosition (QToolButton::BesideIcon);

trunk/src/VBox/Frontends/VirtualBox4/src/VBoxSelectorWnd.cpp

@@ -194 +194 @@
     {
         mRefreshButton->setText (mRefreshAction->text());
-        /* If we set the combination of the text label and icon
-         * below, we lose the Alt+R shortcut functionality... */
-        mRefreshButton->setText (mRefreshAction->text());
-        mRefreshButton->setShortcut (mRefreshAction->shortcut());
         mRefreshButton->setIcon (mRefreshAction->icon());
         mRefreshButton->setToolButtonStyle (Qt::ToolButtonTextBesideIcon);
@@ -417 +413 @@
     trayShowWindowAction->setIcon (VBoxGlobal::iconSet (":/VirtualBox_16px.png"));
     trayShowWindowAction->setText (tr ("Sun xVM VirtualBox"));
-    
+
     trayExitAction = new QAction (this);
     trayExitAction->setMenuRole (QAction::QuitRole);