Changeset 1177 in vbox

Timestamp:

Mar 4, 2007 9:05:35 AM (18 years ago)

Author:

vboxsync

Message:

1848: Add comment field
Issue resolved as designed

Location:

trunk/src/VBox/Frontends/VirtualBox

Files:

• Makefile (modified) (1 diff)
• include/VBoxSelectorWnd.h (modified) (5 diffs)
• src/VBoxSelectorWnd.cpp (modified) (13 diffs)
• ui/VBoxVMSettingsDlg.ui (modified) (3 diffs)
• ui/VBoxVMSettingsDlg.ui.h (modified) (3 diffs)

trunk/src/VBox/Frontends/VirtualBox/Makefile

@@ -74 +74 @@
 # Sources containing local definitions of classes that use the Q_OBJECT macro
 ifneq ($(BUILD_TARGET),win)
-VirtualBox_QT_MOCSRCS = src/COMDefs.cpp
+VirtualBox_QT_MOCSRCS = src/COMDefs.cpp \
+                                                src/VBoxSelectorWnd.cpp
 endif
 

trunk/src/VBox/Frontends/VirtualBox/include/VBoxSelectorWnd.h

@@ -38 +38 @@
 class VBoxSnapshotsWgt;
 class VBoxVMDetailsView;
+class VBoxVMDescriptionPage;
 
 class QLabel;
@@ -64 +65 @@
 
     void vmNew();
-    void vmSettings (const QString &category = QString::null);
+    void vmSettings (const QString &category = QString::null, int = 0);
     void vmDelete();
     void vmStart();
@@ -72 +73 @@
     void refreshVMList();
     void refreshVMItem (const QUuid &aID, bool aDetails,
-                                          bool aSnapshots);
+                                          bool aSnapshots,
+                                          bool aDescription);
 
     void showHelpContents();
@@ -90 +92 @@
 
     void vmListBoxCurrentChanged (bool aRefreshDetails = true,
-                                  bool aRefreshSnapshots = true);
+                                  bool aRefreshSnapshots = true,
+                                  bool aRefreshDescription = true);
     void mediaEnumFinished (const VBoxMediaList &);
 
@@ -123 +126 @@
     VBoxVMDetailsView *vmDetailsView;
     VBoxSnapshotsWgt *vmSnapshotsWgt;
+    VBoxVMDescriptionPage *vmDescriptionPage;
 
     QValueList <CSession> sessions;

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

@@ -194 +194 @@
         mRefreshButton->setUsesTextLabel (true);
     }
+}
+
+// VBoxVMDescriptionPage class
+////////////////////////////////////////////////////////////////////////////////
+
+/**
+ *  Comments page widget to represent VM comments.
+ */
+class VBoxVMDescriptionPage : public QWidget
+{
+    Q_OBJECT
+
+public:
+
+    VBoxVMDescriptionPage (VBoxSelectorWnd *, const char *);
+    ~VBoxVMDescriptionPage() {}
+
+    void setMachine (const CMachine &);
+
+    void languageChange();
+
+private slots:
+
+    void goToSettings();
+
+private:
+
+    VBoxSelectorWnd *mParent;
+    QToolButton *mBtnEdit;
+    QTextBrowser *mBrowser;
+};
+
+VBoxVMDescriptionPage::VBoxVMDescriptionPage (VBoxSelectorWnd *aParent,
+                                              const char *aName)
+    : QWidget (aParent, aName), mParent (aParent)
+    , mBtnEdit (0), mBrowser (0)
+{
+    /* main layout creation */
+    QVBoxLayout *mainLayout = new QVBoxLayout (this, 10, 10, "mainLayout");
+    /* mBrowser creation */
+    mBrowser = new QTextBrowser (this, "mBrowser");
+    mBrowser->setSizePolicy (QSizePolicy::Expanding, QSizePolicy::Expanding);
+    mBrowser->setFocusPolicy (QWidget::StrongFocus);
+    mBrowser->setLinkUnderline (false);
+    mBrowser->setFrameShape (QFrame::NoFrame);
+    mBrowser->setPaper (backgroundBrush());
+    mainLayout->addWidget (mBrowser);
+    /* button layout creation */
+    QHBoxLayout *btnLayout = new QHBoxLayout (mainLayout, 10, "btnLayout");
+    QSpacerItem *spacer = new QSpacerItem (0, 0,
+                                           QSizePolicy::Expanding,
+                                           QSizePolicy::Minimum);
+    btnLayout->addItem (spacer);
+    /* button creation */
+    mBtnEdit = new QToolButton (this, "mBtnEdit");
+    mBtnEdit->setFocusPolicy (QWidget::StrongFocus);
+    mBtnEdit->setIconSet (VBoxGlobal::iconSet ("edit_shared_folder_16px.png",
+                          "edit_shared_folder_disabled_16px.png"));
+    mBtnEdit->setTextPosition (QToolButton::BesideIcon);
+    mBtnEdit->setUsesTextLabel (true);
+    connect (mBtnEdit, SIGNAL (clicked()), this, SLOT (goToSettings()));
+    btnLayout->addWidget (mBtnEdit);
+
+    /* apply language settings */
+    languageChange();
+}
+
+void VBoxVMDescriptionPage::setMachine (const CMachine &aMachine)
+{
+    mBrowser->setText (aMachine.GetDescription());
+}
+
+void VBoxVMDescriptionPage::languageChange()
+{
+    mBtnEdit->setTextLabel (tr ("Edit"));
+    mBtnEdit->setAccel (QString ("Ctrl+E"));
+}
+
+void VBoxVMDescriptionPage::goToSettings()
+{
+    mParent->vmSettings ("#general", 2);
 }
 
@@ -308 +389 @@
                          QString::null);
 
+    /* VM comments page */
+    vmDescriptionPage = new VBoxVMDescriptionPage (this, "vmDescriptionPage");
+    vmTabWidget->addTab (vmDescriptionPage,
+                         VBoxGlobal::iconSet ("edit_shared_folder_16px.png"),
+                         QString::null);
+
     /* add actions to the toolbar */
 
@@ -526 +613 @@
  *      VBoxVMSettingsDlg::setup().
  */
-void VBoxSelectorWnd::vmSettings (const QString &category /* = QString::null */)
+void VBoxSelectorWnd::vmSettings (const QString &category, int aSubPage)
 {
     VBoxVMListBoxItem *item = (VBoxVMListBoxItem *) vmListBox->selectedItem();
@@ -543 +630 @@
     VBoxVMSettingsDlg dlg (this, "VBoxVMSettingsDlg");
     dlg.getFromMachine (m);
-    dlg.setup (category);
+    dlg.setup (category, aSubPage);
 
     if (dlg.exec() == QDialog::Accepted)
@@ -736 +823 @@
     AssertMsgReturn (item, ("Item must be always selected here"), (void) 0);
 
-    refreshVMItem (item->id(), true /* aDetails */, true /* aSnapshots */);
+    refreshVMItem (item->id(), true, true, true);
 }
 
@@ -746 +833 @@
 
 void VBoxSelectorWnd::refreshVMItem (const QUuid &aID, bool aDetails,
-                                                       bool aSnapshots)
+                                                       bool aSnapshots,
+                                                       bool aDescription)
 {
     vmListBox->refresh (aID);
     VBoxVMListBoxItem *item = (VBoxVMListBoxItem *) vmListBox->selectedItem();
     if (item && item->id() == aID)
-        vmListBoxCurrentChanged (aDetails, aSnapshots);
+        vmListBoxCurrentChanged (aDetails, aSnapshots, aDescription);
 }
 
@@ -822 +910 @@
     vmTabWidget->changeTab (vmDetailsView, tr ("&Details"));
     vmTabWidget->changeTab (vmSnapshotsWgt, tr ("&Snapshots"));
+    vmTabWidget->changeTab (vmDescriptionPage, tr ("Des&cription"));
 
     /* ensure the details and screenshot view are updated */
@@ -893 +982 @@
 
     vmDetailsView->languageChange();
+    vmDescriptionPage->languageChange();
 }
 
@@ -900 +990 @@
 
 void VBoxSelectorWnd::vmListBoxCurrentChanged (bool aRefreshDetails,
-                                               bool aRefreshSnapshots)
+                                               bool aRefreshSnapshots,
+                                               bool aRefreshDescription)
 {
     if ( !vmListBox->selectedItem() && vmListBox->currentItem() >= 0 )
@@ -931 +1022 @@
             vmSnapshotsWgt->setMachine (m);
         }
+        if (aRefreshDescription)
+        {
+            vmDescriptionPage->setMachine (m);
+        }
 
         CEnums::MachineState state = item->state();
@@ -939 +1034 @@
         vmDetailsView->setEnabled (modifyEnabled);
         vmSnapshotsWgt->setEnabled (!running);
+        vmDescriptionPage->setEnabled (!running);
 
         /* enable/disable modify actions */
@@ -1041 +1137 @@
 void VBoxSelectorWnd::machineStateChanged (const VBoxMachineStateChangeEvent &e)
 {
-    refreshVMItem (e.id, false /* aDetails */, false /* aSnapshots */);
+    refreshVMItem (e.id, false, false, false);
 }
 
 void VBoxSelectorWnd::machineDataChanged (const VBoxMachineDataChangeEvent &e)
 {
-    refreshVMItem (e.id, true /* aDetails */, false /* aSnapshots */);
+    refreshVMItem (e.id, true, false, true);
 }
 
@@ -1075 +1171 @@
 void VBoxSelectorWnd::sessionStateChanged (const VBoxSessionStateChangeEvent &e)
 {
-    refreshVMItem (e.id, false /* aDetails */, false /* aSnapshots */);
+    refreshVMItem (e.id, false, false, false);
 }
 
 void VBoxSelectorWnd::snapshotChanged (const VBoxSnapshotEvent &aEvent)
 {
-    refreshVMItem (aEvent.machineId, false, true);
-}
+    refreshVMItem (aEvent.machineId, false, true, false);
+}
+
+#include "VBoxSelectorWnd.moc"

trunk/src/VBox/Frontends/VirtualBox/ui/VBoxVMSettingsDlg.ui

@@ -1182 +1182 @@
                             </grid>
                         </widget>
+                        <widget class="QWidget">
+                            <property name="name">
+                                <cstring>description</cstring>
+                            </property>
+                            <attribute name="title">
+                                <string>&amp;Description</string>
+                            </attribute>
+                            <vbox>
+                                <property name="name">
+                                    <cstring>unnamed</cstring>
+                                </property>
+                                <property name="margin">
+                                    <number>10</number>
+                                </property>
+                                <property name="spacing">
+                                    <number>10</number>
+                                </property>
+                                <widget class="QGroupBox">
+                                    <property name="name">
+                                        <cstring>gbDescription</cstring>
+                                    </property>
+                                    <property name="title">
+                                        <string>Virtual Machine &amp;Description</string>
+                                    </property>
+                                    <vbox>
+                                        <property name="name">
+                                            <cstring>unnamed</cstring>
+                                        </property>
+                                        <widget class="QTextEdit">
+                                            <property name="name">
+                                                <cstring>teDescription</cstring>
+                                            </property>
+                                        </widget>
+                                    </vbox>
+                                </widget>
+                            </vbox>
+                        </widget>
                     </widget>
                 </vbox>
@@ -1804 +1841 @@
                                 </property>
                                 <property name="whatsThis" stdset="0">
-                                    <string>Controls the audio output driver. The &lt;b&gt;Null Audio Driver&lt;/b&gt; 
+                                    <string>Controls the audio output driver. The &lt;b&gt;Null Audio Driver&lt;/b&gt;
 makes the guest see an audio card, however every access to it will be ignored.</string>
                                 </property>
@@ -2722 +2759 @@
 <functions>
     <function access="private">init()</function>
-    <function>setup( const QString &amp; category )</function>
+    <function>setup( const QString &amp;, int )</function>
     <function>getFromMachine( const CMachine &amp; machine )</function>
     <function returnType="COMResult">putBackToMachine()</function>

trunk/src/VBox/Frontends/VirtualBox/ui/VBoxVMSettingsDlg.ui.h

@@ -822 +822 @@
  *      sign.
  */
-void VBoxVMSettingsDlg::setup (const QString &category)
-{
-    if (category)
-    {
-        QListViewItem *item = listView->findItem (category, listView_Link);
+void VBoxVMSettingsDlg::setup (const QString &aCategory, int aSubPage)
+{
+    if (!aCategory.isNull())
+    {
+        QListViewItem *item = listView->findItem (aCategory, listView_Link);
         if (item)
+        {
             listView->setSelected (item, true);
+            QObjectList *list = widgetStack->visibleWidget()->queryList ("QTabWidget");
+            for (QObject *obj = list->first(); obj != NULL; obj = list->next())
+            {
+                QTabWidget *tabStack = static_cast<QTabWidget*> (obj);
+                tabStack->setCurrentPage (aSubPage);
+            }
+        }
     }
 }
@@ -1066 +1074 @@
     /* Saved state folder */
     leSnapshotFolder->setText (machine.GetSnapshotFolder());
+
+    /* Description */
+    teDescription->setText (machine.GetDescription());
 
     /* hard disk images */
@@ -1380 +1391 @@
         cmachine.SetSnapshotFolder (leSnapshotFolder->text());
 
+    /* Description */
+    cmachine.SetDescription (teDescription->text());
+
     /* hard disk images */
     {