[freetype2-demos] gsoc-2022-chariri-3 38f60d8 08/36: [ftinspect] Add "Show Grid" check box.

[Werner Lemberg]

branch: gsoc-2022-chariri-3
commit 38f60d85c5d882c600b3addfc683fa9c81207514
Author: Charlie Jiang <w@chariri.moe>
Commit: Charlie Jiang <w@chariri.moe>

    [ftinspect] Add "Show Grid" check box.
    
    * src/ftinspect/panels/singular.cpp, src/ftinspect/panels/singular.hpp:
      Add "Show Grid" check box, which uses `QGraphicsItem::setVisible`.
    
    * src/ftinspect/rendering/grid.cpp: Remove TODO that has been resolved.
---
 src/ftinspect/panels/singular.cpp | 12 ++++++++++++
 src/ftinspect/panels/singular.hpp |  2 ++
 src/ftinspect/rendering/grid.cpp  |  3 ---
 3 files changed, 14 insertions(+), 3 deletions(-)

diff --git a/src/ftinspect/panels/singular.cpp 
b/src/ftinspect/panels/singular.cpp
index 0fb50a6..2c824d4 100644
--- a/src/ftinspect/panels/singular.cpp
+++ b/src/ftinspect/panels/singular.cpp
@@ -197,6 +197,13 @@ SingularTab::wheelResize(QWheelEvent* event)
 }
 
 
+void
+SingularTab::setGridVisible()
+{
+  gridItem_->setVisible(showGridCheckBox_->isChecked());
+}
+
+
 void
 SingularTab::createLayout()
 {
@@ -261,6 +268,7 @@ SingularTab::createLayout()
   showPointsCheckBox_ = new QCheckBox(tr("Show Points"), this);
   showPointNumbersCheckBox_ = new QCheckBox(tr("Show Point Numbers"), this);
   showOutlinesCheckBox_ = new QCheckBox(tr("Show Outlines"), this);
+  showGridCheckBox_ = new QCheckBox(tr("Show Grid"), this);
 
   sizeLayout_ = new QHBoxLayout;
   sizeLayout_->addStretch(2);
@@ -278,6 +286,7 @@ SingularTab::createLayout()
   checkBoxesLayout_->addWidget(showPointsCheckBox_);
   checkBoxesLayout_->addWidget(showPointNumbersCheckBox_);
   checkBoxesLayout_->addWidget(showOutlinesCheckBox_);
+  checkBoxesLayout_->addWidget(showGridCheckBox_);
 
   glyphOverlayIndexLayout_ = new QHBoxLayout;
   glyphOverlayIndexLayout_->addWidget(glyphIndexLabel_);
@@ -331,6 +340,8 @@ SingularTab::createConnections()
           this, &SingularTab::drawGlyph);
   connect(showOutlinesCheckBox_, &QCheckBox::clicked,
           this, &SingularTab::drawGlyph);
+  connect(showGridCheckBox_, &QCheckBox::clicked,
+          this, &SingularTab::setGridVisible);
 }
 
 
@@ -365,6 +376,7 @@ SingularTab::setDefaults()
   zoomSpinBox_->setValue(20);
   showBitmapCheckBox_->setChecked(true);
   showOutlinesCheckBox_->setChecked(true);
+  showGridCheckBox_->setChecked(true);
   
   indexSelector_->setCurrentIndex(indexSelector_->currentIndex(), true);
   zoom();
diff --git a/src/ftinspect/panels/singular.hpp 
b/src/ftinspect/panels/singular.hpp
index 887cd30..6c83535 100644
--- a/src/ftinspect/panels/singular.hpp
+++ b/src/ftinspect/panels/singular.hpp
@@ -52,6 +52,7 @@ private slots:
   void backToCenter();
   void wheelZoom(QWheelEvent* event);
   void wheelResize(QWheelEvent* event);
+  void setGridVisible();
 
 private:
   int currentGlyphIndex_;
@@ -82,6 +83,7 @@ private:
   QCheckBox* showOutlinesCheckBox_;
   QCheckBox* showPointNumbersCheckBox_;
   QCheckBox* showPointsCheckBox_;
+  QCheckBox* showGridCheckBox_;
 
   QVBoxLayout* mainLayout_;
   QHBoxLayout* checkBoxesLayout_;
diff --git a/src/ftinspect/rendering/grid.cpp b/src/ftinspect/rendering/grid.cpp
index 875afc0..24cabe2 100644
--- a/src/ftinspect/rendering/grid.cpp
+++ b/src/ftinspect/rendering/grid.cpp
@@ -55,9 +55,6 @@ Grid::updateRect()
 }
 
 
-// XXX call this in a `myQDraphicsView::drawBackground' derived method
-//     to always fill the complete viewport
-
 void
 Grid::paint(QPainter* painter,
             const QStyleOptionGraphicsItem* option,