Expand the status bar info for the Eraser (#4261)

When erasing a rectangle, now shows "Erase area (<width> x <height>)".

Related to issue #4201

Author: bjorn

NEWS.md

@@ -5,7 +5,7 @@
 * Added support for SVG 1.2 / CSS blending modes to layers (#3932)
 * Added button to toggle Terrain Brush to full tile mode (by Finlay Pearson, #3407)
 * Added square selection and expand-from-center to Rectangular Select tool (#4201)
-* Added status bar info for various Stamp and Terrain Brush modes (#3092, #4201)
+* Added status info for various Stamp Brush, Terrain Brush and Eraser modes (#3092, #4201)
 * Added export plugin for Remixed Dungeon (by Mikhael Danilov, #4158)
 * Added "World > World Properties" menu action (with dogboydog, #4190)
 * Scripting: Added API for custom property types (with dogboydog, #3971)

src/tiled/eraser.cpp

@@ -57,10 +57,14 @@ void Eraser::mousePressed(QGraphicsSceneMouseEvent *event)
         if (event->button() == Qt::LeftButton) {
             mMode = Erase;
             doErase(false);
+            updateStatusInfo();
             return;
-        } else if (event->button() == Qt::RightButton && !(event->modifiers() & Qt::ControlModifier)) {
+        }
+
+        if (event->button() == Qt::RightButton && !(event->modifiers() & Qt::ControlModifier)) {
             mStart = tilePosition();
             mMode = RectangleErase;
+            updateStatusInfo();
             return;
         }
     }
@@ -82,6 +86,7 @@ void Eraser::mouseReleased(QGraphicsSceneMouseEvent *event)
             doErase(false);
             mMode = Nothing;
             brushItem()->setTileRegion(eraseArea());
+            updateStatusInfo();
         }
         break;
     }
@@ -130,4 +135,25 @@ QRect Eraser::eraseArea() const
     return QRect(tilePosition(), QSize(1, 1));
 }
 
+void Eraser::updateStatusInfo()
+{
+    if (!isBrushVisible()) {
+        AbstractTileTool::updateStatusInfo();
+        return;
+    }
+
+    if (mMode == RectangleErase) {
+        const QPoint pos = tilePosition();
+        const QRect rect = eraseArea();
+        setStatusInfo(tr("%1, %2 - Erase area (%4 x %5)")
+                          .arg(pos.x())
+                          .arg(pos.y())
+                          .arg(rect.width())
+                          .arg(rect.height()));
+        return;
+    }
+
+    AbstractTileTool::updateStatusInfo();
+}
+
 #include "moc_eraser.cpp"

src/tiled/eraser.h

@@ -43,6 +43,7 @@ class Eraser : public AbstractTileTool
 
 protected:
     void tilePositionChanged(QPoint tilePos) override;
+    void updateStatusInfo() override;
 
 private:
     void doErase(bool continuation);