Board editor: Allow to hide copper planes

img/images.qrc

@@ -37,6 +37,7 @@
         <file>actions/generate_bom.png</file>
         <file>actions/grid.png</file>
         <file>actions/help.png</file>
+        <file>actions/hide_planes.png</file>
         <file>actions/home.png</file>
         <file>actions/info.png</file>
         <file>actions/layers.png</file>
@@ -67,6 +68,7 @@
         <file>actions/select.png</file>
         <file>actions/select_rect.png</file>
         <file>actions/settings.png</file>
+        <file>actions/show_planes.png</file>
         <file>actions/stop.png</file>
         <file>actions/undo.png</file>
         <file>actions/up.png</file>

libs/librepcb/project/boards/graphicsitems/bgi_plane.cpp

@@ -110,10 +110,12 @@ void BGI_Plane::paint(QPainter* painter, const QStyleOptionGraphicsItem* option,
     painter->setBrush(Qt::NoBrush);
     painter->drawPath(mOutline);
 
-    // draw plane
-    painter->setPen(Qt::NoPen);
-    painter->setBrush(mLayer->getColor(selected));
-    foreach (const QPainterPath& area, mAreas) { painter->drawPath(area); }
+    // draw plane only if plane should be visible
+    if (mPlane.isVisible()) {
+      painter->setPen(Qt::NoPen);
+      painter->setBrush(mLayer->getColor(selected));
+      foreach (const QPainterPath& area, mAreas) { painter->drawPath(area); }
+    }
   }
 
 #ifdef QT_DEBUG

libs/librepcb/project/boards/items/bi_plane.cpp

@@ -55,6 +55,7 @@ BI_Plane::BI_Plane(Board& board, const BI_Plane& other)
     mConnectStyle(other.mConnectStyle),
     // mThermalGapWidth(other.mThermalGapWidth),
     // mThermalSpokeWidth(other.mThermalSpokeWidth),
+    mIsVisible(true),
     mFragments(other.mFragments)  // also copy fragments to avoid the need for
                                   // a rebuild
 {
@@ -71,10 +72,11 @@ BI_Plane::BI_Plane(Board& board, const SExpression& node)
     mMinClearance(node.getValueByPath<UnsignedLength>("min_clearance")),
     mKeepOrphans(node.getValueByPath<bool>("keep_orphans")),
     mPriority(node.getValueByPath<int>("priority")),
-    mConnectStyle(node.getValueByPath<ConnectStyle>("connect_style"))
-// mThermalGapWidth(node.getValueByPath<Length>("thermal_gap_width", true)),
-// mThermalSpokeWidth(node.getValueByPath<Length>("thermal_spoke_width", true))
-{
+    mConnectStyle(node.getValueByPath<ConnectStyle>("connect_style")),
+    // mThermalGapWidth(node.getValueByPath<Length>("thermal_gap_width", true)),
+    // mThermalSpokeWidth(node.getValueByPath<Length>("thermal_spoke_width",
+    // true))
+    mIsVisible(true) {
   Uuid netSignalUuid = node.getValueByPath<Uuid>("net");
   mNetSignal =
       mBoard.getProject().getCircuit().getNetSignalByUuid(netSignalUuid);
@@ -101,6 +103,7 @@ BI_Plane::BI_Plane(Board& board, const Uuid& uuid,
     mPriority(0),
     mConnectStyle(ConnectStyle::Solid),
     // mThermalGapWidth(100000), mThermalSpokeWidth(100000),
+    mIsVisible(true),
     mFragments() {
   init();
 }
@@ -182,6 +185,13 @@ void BI_Plane::setKeepOrphans(bool keepOrphans) noexcept {
   }
 }
 
+void BI_Plane::setVisible(bool visible) noexcept {
+  if (visible != mIsVisible) {
+    mIsVisible = visible;
+    mGraphicsItem->update();
+  }
+}
+
 /*******************************************************************************
  *  General Methods
  ******************************************************************************/

libs/librepcb/project/boards/items/bi_plane.h

@@ -87,6 +87,7 @@ class BI_Plane final : public BI_Base, public SerializableObject {
   const Path&          getOutline() const noexcept { return mOutline; }
   const QVector<Path>& getFragments() const noexcept { return mFragments; }
   bool                 isSelectable() const noexcept override;
+  bool                 isVisible() const noexcept { return mIsVisible; }
 
   // Setters
   void setOutline(const Path& outline) noexcept;
@@ -97,6 +98,7 @@ class BI_Plane final : public BI_Base, public SerializableObject {
   void setConnectStyle(ConnectStyle style) noexcept;
   void setPriority(int priority) noexcept;
   void setKeepOrphans(bool keepOrphans) noexcept;
+  void setVisible(bool visible) noexcept;
 
   // General Methods
   void addToBoard() override;
@@ -142,6 +144,7 @@ private slots:
   // Length mThermalSpokeWidth;
   // style [round square miter] ?
   QScopedPointer<BGI_Plane> mGraphicsItem;
+  bool                      mIsVisible;  // volatile, not saved to file
 
   QVector<Path> mFragments;
 };

libs/librepcb/projecteditor/boardeditor/boardeditor.cpp

@@ -531,6 +531,22 @@ void BoardEditor::on_actionRebuildPlanes_triggered() {
   }
 }
 
+void BoardEditor::on_actionShowAllPlanes_triggered() {
+  if (Board* board = getActiveBoard()) {
+    foreach (BI_Plane* plane, board->getPlanes()) {
+      plane->setVisible(true);  // No undo command needed since it is not saved
+    }
+  }
+}
+
+void BoardEditor::on_actionHideAllPlanes_triggered() {
+  if (Board* board = getActiveBoard()) {
+    foreach (BI_Plane* plane, board->getPlanes()) {
+      plane->setVisible(false);  // No undo command needed since it is not saved
+    }
+  }
+}
+
 void BoardEditor::on_tabBar_currentChanged(int index) {
   setActiveBoardIndex(index);
 }

libs/librepcb/projecteditor/boardeditor/boardeditor.h

@@ -108,6 +108,8 @@ private slots:
   void on_actionLayerStackSetup_triggered();
   void on_actionModifyDesignRules_triggered();
   void on_actionRebuildPlanes_triggered();
+  void on_actionShowAllPlanes_triggered();
+  void on_actionHideAllPlanes_triggered();
   void on_tabBar_currentChanged(int index);
   void on_lblUnplacedComponentsNote_linkActivated();
   void boardListActionGroupTriggered(QAction* action);

libs/librepcb/projecteditor/boardeditor/boardeditor.ui

@@ -112,6 +112,9 @@
     </property>
     <addaction name="actionGrid"/>
     <addaction name="separator"/>
+    <addaction name="actionHideAllPlanes"/>
+    <addaction name="actionShowAllPlanes"/>
+    <addaction name="separator"/>
     <addaction name="actionZoomIn"/>
     <addaction name="actionZoomOut"/>
     <addaction name="actionZoomAll"/>
@@ -760,6 +763,24 @@
     <string>Generate BOM</string>
    </property>
   </action>
+  <action name="actionShowAllPlanes">
+   <property name="icon">
+    <iconset resource="../../../../img/images.qrc">
+     <normaloff>:/img/actions/show_planes.png</normaloff>:/img/actions/show_planes.png</iconset>
+   </property>
+   <property name="text">
+    <string>Show All Planes</string>
+   </property>
+  </action>
+  <action name="actionHideAllPlanes">
+   <property name="icon">
+    <iconset resource="../../../../img/images.qrc">
+     <normaloff>:/img/actions/hide_planes.png</normaloff>:/img/actions/hide_planes.png</iconset>
+   </property>
+   <property name="text">
+    <string>Hide All Planes</string>
+   </property>
+  </action>
  </widget>
  <customwidgets>
   <customwidget>

libs/librepcb/projecteditor/boardeditor/fsm/bes_select.cpp

@@ -444,6 +444,10 @@ BES_Base::ProcRetVal BES_Select::proccessIdleSceneRightMouseButtonReleased(
       QAction* aRemove =
           menu.addAction(QIcon(":/img/actions/delete.png"), "Remove Plane");
       menu.addSeparator();
+      QAction* aIsVisible = menu.addAction(tr("Visible"));
+      aIsVisible->setCheckable(true);
+      aIsVisible->setChecked(plane->isVisible());
+      menu.addSeparator();
       QAction* aProperties = menu.addAction(QIcon(":/img/actions/settings.png"),
                                             tr("Plane Properties"));
 
@@ -457,6 +461,9 @@ BES_Base::ProcRetVal BES_Select::proccessIdleSceneRightMouseButtonReleased(
         flipSelectedItems(Qt::Horizontal);
       } else if (action == aRemove) {
         removeSelectedItems();
+      } else if (action == aIsVisible) {
+        // Visibility is not saved, thus no undo command is needed here.
+        plane->setVisible(aIsVisible->isChecked());
       } else if (action == aProperties) {
         openPlanePropertiesDialog(*plane);
       }
@@ -750,7 +757,16 @@ void BES_Select::openViaPropertiesDialog(BI_Via& via) noexcept {
 
 void BES_Select::openPlanePropertiesDialog(BI_Plane& plane) noexcept {
   BoardPlanePropertiesDialog dialog(mProject, plane, mUndoStack, &mEditor);
+
+  // Make sure the plane is visible visible since it's useful to see the actual
+  // plane fragments while the plane properties are modified.
+  bool visible = plane.isVisible();
+  plane.setVisible(true);
+
   dialog.exec();
+
+  // Restore visibility
+  plane.setVisible(visible);
 }
 
 void BES_Select::openPolygonPropertiesDialog(Board&   board,