debugger: Make inspector widget smaller

debugger/QT/SkInspectorWidget.cpp

Index: debugger/QT/SkInspectorWidget.cpp
diff --git a/debugger/QT/SkInspectorWidget.cpp b/debugger/QT/SkInspectorWidget.cpp
index 6bcac1db357bf592f4212c3508d02de5479552f9..50fe70a02078eec11c1acdabb4b819b0122d8da8 100644
--- a/debugger/QT/SkInspectorWidget.cpp
+++ b/debugger/QT/SkInspectorWidget.cpp
@@ -10,16 +10,14 @@
 #include "SkInspectorWidget.h"
 #include <iostream>
 

[robertphillips, 2014/12/31 21:06:52]

static const int ?

[Kimmo Kinnunen, 2015/01/02 07:17:54]

Done.

+enum {
+    kSignificantNumbersInFields = 5
+};
+
 SkInspectorWidget::SkInspectorWidget() : QWidget()
     , fHorizontalLayout(this)
     , fMatrixAndClipWidget(this)
-    , fVerticalLayout(&fMatrixAndClipWidget)
-    , fMatrixLabel(this)
-    , fClipLabel(this) {
-
-    fHorizontalLayout.setSpacing(6);
-    fHorizontalLayout.setContentsMargins(11, 11, 11, 11);
-
+    , fVerticalLayout(&fMatrixAndClipWidget) {
     QString tabNames[kTotalTabCount];
     tabNames[kOverview_TabType] = "Overview";
     tabNames[kDetail_TabType] = "Details";
@@ -27,8 +25,6 @@ SkInspectorWidget::SkInspectorWidget() : QWidget()
 
     for (int i = 0; i < kTotalTabCount; i++) {
         fTabTexts[i].setReadOnly(true);
-        fTabLayouts[i].setSpacing(6);
-        fTabLayouts[i].setContentsMargins(11, 11, 11, 11);
         fTabLayouts[i].addWidget(&fTabTexts[i]);
         fTabs[i].setLayout(&fTabLayouts[i]);
         fTabWidget.addTab(&fTabs[i], tabNames[i]);
@@ -37,17 +33,13 @@ SkInspectorWidget::SkInspectorWidget() : QWidget()
     fHorizontalLayout.setAlignment(Qt::AlignTop);
     fHorizontalLayout.addWidget(&fTabWidget);
 
-    /* NOTE(chudy): We add all the line edits to (this). Then we lay them out
-     * by adding them to horizontal layouts.
-     *
-     * We will have 1 big vertical layout, 3 horizontal layouts and then 3
-     * line edits in each horizontal layout. */
-    fMatrixAndClipWidget.setFixedSize(260,300);
+    fMatrixAndClipWidget.setFrameStyle(QFrame::Panel);
     fMatrixAndClipWidget.setDisabled(true);
-
     fVerticalLayout.setAlignment(Qt::AlignVCenter);

[robertphillips, 2014/12/31 21:06:53]

this-> on these two ?

[Kimmo Kinnunen, 2015/01/02 07:17:54]

Done.

-    fVerticalLayout.addLayout(setupMatrix());
-    fVerticalLayout.addLayout(setupClip());
+    setupMatrix();
+    setupClip();
+    fVerticalLayout.addWidget(&fMatrixGroup);
+    fVerticalLayout.addWidget(&fClipGroup);
     fHorizontalLayout.addWidget(&fMatrixAndClipWidget);
 }
 
@@ -57,59 +49,39 @@ void SkInspectorWidget::setText(QString text, TabType type) {
 
 void SkInspectorWidget::setMatrix(const SkMatrix& matrix) {
     for(int i=0; i<9; i++) {
-        fMatrixEntry[i].setText(QString::number(matrix.get(i)));
+        fMatrixEntry[i].setText(QString::number(matrix.get(i), 'g', kSignificantNumbersInFields));
     }
 }
 
 void SkInspectorWidget::setClip(const SkIRect& clip) {
-    fClipEntry[0].setText(QString::number(clip.left()));
-    fClipEntry[1].setText(QString::number(clip.top()));
-    fClipEntry[2].setText(QString::number(clip.right()));
-    fClipEntry[3].setText(QString::number(clip.bottom()));
+    fClipEntry[0].setText(QString::number(clip.left(), 'g', kSignificantNumbersInFields));
+    fClipEntry[1].setText(QString::number(clip.top(), 'g', kSignificantNumbersInFields));
+    fClipEntry[2].setText(QString::number(clip.right(), 'g', kSignificantNumbersInFields));
+    fClipEntry[3].setText(QString::number(clip.bottom(), 'g', kSignificantNumbersInFields));
 }
 
-QVBoxLayout* SkInspectorWidget::setupMatrix() {
-    fMatrixLabel.setText("Current Matrix");
-    fMatrixLabel.setAlignment(Qt::AlignHCenter);
-
-    fMatrixLayout.setSpacing(6);
-    fMatrixLayout.setContentsMargins(11,11,11,0);
-    fMatrixLayout.setAlignment(Qt::AlignTop | Qt::AlignHCenter);
-    fMatrixLayout.addWidget(&fMatrixLabel);
-
-    for(int i =0; i<9; i++) {
-        fMatrixEntry[i].setMinimumSize(QSize(70,25));
-        fMatrixEntry[i].setMaximumSize(QSize(70,16777215));
-        fMatrixEntry[i].setReadOnly(true);
-
-        fMatrixRow[i/3].addWidget(&fMatrixEntry[i]);
-        if(i%3 == 2) {
-            fMatrixLayout.addLayout(&fMatrixRow[i/3]);
+void SkInspectorWidget::setupMatrix() {
+    fMatrixGroup.setTitle("Current Matrix");
+    fMatrixGroup.setLayout(&fMatrixLayout);
+    for (int r = 0; r < 3; ++r) {
+        for(int c = 0; c < 3; c++) {
+            QLineEdit* entry = &fMatrixEntry[r * 3 + c];
+            fMatrixLayout.addWidget(entry, r, c, Qt::AlignTop | Qt::AlignHCenter);
+            entry->setReadOnly(true);
+            entry->setFixedWidth(70);
         }
     }
-
-    return &fMatrixLayout;
 }
 
-QVBoxLayout* SkInspectorWidget::setupClip() {
-    fClipLabel.setText("Current Clip");
-    fClipLabel.setAlignment(Qt::AlignHCenter);
-
-    fClipLayout.setSpacing(6);
-    fClipLayout.setContentsMargins(11,11,11,0);
-    fClipLayout.setAlignment(Qt::AlignTop | Qt::AlignHCenter);
-    fClipLayout.addWidget(&fClipLabel);
-
-    for(int i =0; i<4; i++) {
-        fClipEntry[i].setMinimumSize(QSize(70,25));
-        fClipEntry[i].setMaximumSize(QSize(70,16777215));
-        fClipEntry[i].setReadOnly(true);
-
-        fClipRow[i/2].addWidget(&fClipEntry[i]);
-        if(i%2 == 1) {
-            fClipLayout.addLayout(&fClipRow[i/2]);
+void SkInspectorWidget::setupClip() {
+    fClipGroup.setTitle("Current Clip");
+    fClipGroup.setLayout(&fClipLayout);
+    for(int r = 0; r < 2; r++) {
+        for(int c = 0; c < 2; c++) {
+            QLineEdit* entry = &fClipEntry[r * 2 + c];
+            fClipLayout.addWidget(entry, r, c, Qt::AlignTop | Qt::AlignHCenter);
+            entry->setReadOnly(true);
+            entry->setFixedWidth(70);
         }
     }
-
-    return &fClipLayout;
 }