Changeset 88272 in vbox

Timestamp:

Mar 24, 2021 12:06:05 PM (4 years ago)

Author:

vboxsync

Message:

FE/Qt: bugref:9515. Some more code style fixes

File:

• trunk/src/VBox/Frontends/VirtualBox/src/widgets/UIToolBox.cpp (modified) (2 diffs)

trunk/src/VBox/Frontends/VirtualBox/src/widgets/UIToolBox.cpp

@@ -31 +31 @@
 #include "UIWizardNewVM.h"
 
+/*********************************************************************************************************************************
+*   UIToolPageButton definition.                                                                                    *
+*********************************************************************************************************************************/
+/** A QAbstractButton extension used to show collapse/expand icons. More importantly
+  * it is buddy to the title label which may include some mnemonics. This makes it possible
+  * to expand pages via keyboard. */
 class UIToolPageButton : public QAbstractButton
 {
+
     Q_OBJECT;
+
 public:
+
     UIToolPageButton(QWidget *pParent = 0);
     void setPixmap(const QPixmap &pixmap);
+
 protected:
+
     virtual void paintEvent(QPaintEvent *pEvent) /* override */;
     virtual QSize sizeHint() const /* override */;
+
 private:
+
+    /** Holds the pixmap of the expand/collapser icon. We keep
+      * QPixmap instead of QIcon since it is rotated when the
+      * the page is expanded and end product of rotation is a pixmap.
+      * and we use QPainter to draw pixmap.*/
     QPixmap m_pixmap;
 };
-
-UIToolPageButton::UIToolPageButton(QWidget *pParent /* = 0 */)
-    :QAbstractButton(pParent)
-{
-}
-
-void UIToolPageButton::paintEvent(QPaintEvent *pEvent)
-{
-    Q_UNUSED(pEvent);
-    if (!m_pixmap.isNull())
-    {
-        QPainter painter(this);
-        painter.drawPixmap(0, 0, m_pixmap.width(), m_pixmap.height(), m_pixmap);
-    }
-}
-
-void UIToolPageButton::setPixmap(const QPixmap &pixmap)
-{
-    m_pixmap = pixmap;
-    update();
-}
-
-QSize UIToolPageButton::sizeHint() const
-{
-    if (m_pixmap.isNull())
-        return QSize (0,0);
-    return m_pixmap.size();
-}
 
 /*********************************************************************************************************************************
@@ -128 +117 @@
 
 /*********************************************************************************************************************************
+*   UIToolPageButton implementation.                                                                                    *
+*********************************************************************************************************************************/
+
+UIToolPageButton::UIToolPageButton(QWidget *pParent /* = 0 */)
+    :QAbstractButton(pParent)
+{
+}
+
+void UIToolPageButton::paintEvent(QPaintEvent *pEvent)
+{
+    Q_UNUSED(pEvent);
+    if (!m_pixmap.isNull())
+    {
+        QPainter painter(this);
+        painter.drawPixmap(0, 0, m_pixmap.width(), m_pixmap.height(), m_pixmap);
+    }
+}
+
+void UIToolPageButton::setPixmap(const QPixmap &pixmap)
+{
+    m_pixmap = pixmap;
+    update();
+}
+
+QSize UIToolPageButton::sizeHint() const
+{
+    if (m_pixmap.isNull())
+        return QSize (0,0);
+    return m_pixmap.size();
+}
+
+/*********************************************************************************************************************************
 *   UIToolBoxPage implementation.                                                                                    *
 *********************************************************************************************************************************/