Changeset 67261 in vbox for trunk

Timestamp:

Jun 5, 2017 4:17:57 PM (8 years ago)

Author:

vboxsync

Message:

FE/Qt: Selector UI: Tools pane: Snapshot pane: Better encapsulation for 'current snapshot item'.

File:

• trunk/src/VBox/Frontends/VirtualBox/src/selector/UISnapshotPane.cpp (modified) (11 diffs)

trunk/src/VBox/Frontends/VirtualBox/src/selector/UISnapshotPane.cpp

@@ -96 +96 @@
     bool isCurrentStateItem() const { return m_fCurrentStateItem; }
 
+    /** Returns whether this is the current snapshot item. */
+    bool isCurrentSnapshotItem() const { return m_fCurrentSnapshotItem; }
+    /** Defines whether this is the @a fCurrent snapshot item. */
+    void setCurrentSnapshotItem(bool fCurrent);
+
     /** Calculates and returns the current item level. */
     int level() const;
-
-    /** Returns whether the font is bold. */
-    bool bold() const;
-    /** Defines whether the font is @a fBold. */
-    void setBold(bool fBold);
-
-    /** Returns whether the font is italic. */
-    bool italic() const;
-    /** Defines whether the font is @a fItalic. */
-    void setItalic(bool fItalic);
 
     /** Recaches the item's contents. */
@@ -135 +130 @@
     /** Holds whether this is a "current state" item. */
     bool  m_fCurrentStateItem;
+    /** Holds whether this is a "current snapshot" item. */
+    bool  m_fCurrentSnapshotItem;
 
     /** Holds the snapshot COM wrapper. */
@@ -200 +197 @@
     , m_pSnapshotWidget(pSnapshotWidget)
     , m_fCurrentStateItem(false)
+    , m_fCurrentSnapshotItem(false)
     , m_comSnapshot(comSnapshot)
 {
@@ -208 +206 @@
     , m_pSnapshotWidget(pSnapshotWidget)
     , m_fCurrentStateItem(false)
+    , m_fCurrentSnapshotItem(false)
     , m_comSnapshot(comSnapshot)
 {
@@ -216 +215 @@
     , m_pSnapshotWidget(pSnapshotWidget)
     , m_fCurrentStateItem(true)
+    , m_fCurrentSnapshotItem(false)
     , m_comMachine(comMachine)
 {
+    /* Set the bold font state
+     * for current state item: */
+    QFont myFont = font(0);
+    myFont.setBold(true);
+    setFont(0, myFont);
+
     /* Fetch current machine state: */
     setMachineState(m_comMachine.GetState());
@@ -226 +232 @@
     , m_pSnapshotWidget(pSnapshotWidget)
     , m_fCurrentStateItem(true)
+    , m_fCurrentSnapshotItem(false)
     , m_comMachine(comMachine)
 {
+    /* Set the bold font state
+     * for current state item: */
+    QFont myFont = font(0);
+    myFont.setBold(true);
+    setFont(0, myFont);
+
     /* Fetch current machine state: */
     setMachineState(m_comMachine.GetState());
@@ -282 +295 @@
 }
 
-bool UISnapshotItem::bold() const
-{
-    return font(0).bold();
-}
-
-void UISnapshotItem::setBold(bool fBold)
-{
-    /* Update font: */
+void UISnapshotItem::setCurrentSnapshotItem(bool fCurrent)
+{
+    /* Remember the state: */
+    m_fCurrentSnapshotItem = fCurrent;
+
+    /* Set/clear the bold font state
+     * for current snapshot item: */
     QFont myFont = font(0);
-    myFont.setBold(fBold);
+    myFont.setBold(fCurrent);
     setFont(0, myFont);
 
-    /* Adjust text: */
-    adjustText();
-}
-
-bool UISnapshotItem::italic() const
-{
-    return font(0).italic();
-}
-
-void UISnapshotItem::setItalic(bool fItalic)
-{
-    /* Update font: */
-    QFont myFont = font(0);
-    myFont.setItalic(fItalic);
-    setFont(0, myFont);
-
-    /* Adjust text: */
-    adjustText();
+    /* Update tool-tip: */
+    recacheToolTip();
 }
 
@@ -462 +458 @@
     else
     {
-        /* The current snapshot is always bold: */
-        if (bold())
+        if (isCurrentSnapshotItem())
             strDetails = UISnapshotPane::tr(" (current, ", "Snapshot details");
         else
@@ -1131 +1126 @@
         /* Add the "current state" item as a child to current snapshot item: */
         UISnapshotItem *pCsi = new UISnapshotItem(this, m_pCurrentSnapshotItem, m_comMachine);
-        pCsi->setBold(true);
         pCsi->recache();
 
@@ -1154 +1148 @@
         /* Add the "current state" item as a child of snapshot tree: */
         UISnapshotItem *pCsi = new UISnapshotItem(this, m_pSnapshotTree, m_comMachine);
-        pCsi->setBold(true);
         pCsi->recache();
 
@@ -1177 +1170 @@
     pSnapshotItem->recache();
 
-    /* Mark current snapshot item bold and remember it: */
+    /* Mark snapshot item as "current" and remember it: */
     CSnapshot comCurrentSnapshot = m_comMachine.GetCurrentSnapshot();
     if (!comCurrentSnapshot.isNull() && comCurrentSnapshot.GetId() == comSnapshot.GetId())
     {
-        pSnapshotItem->setBold(true);
+        pSnapshotItem->setCurrentSnapshotItem(true);
         m_pCurrentSnapshotItem = pSnapshotItem;
     }