Eraser: Added Shift to erase on all layers (#2897)

Co-authored-by: Thorbjørn Lindeijer <[email protected]>
mapeditor · Sep 29, 2020 · a75e8b9 · a75e8b9
1 parent 10c943f
commit a75e8b9
Show file tree

Hide file tree

Showing 3 changed files with 25 additions and 11 deletions.
diff --git a/docs/manual/editing-tile-layers.rst b/docs/manual/editing-tile-layers.rst
@@ -147,6 +147,8 @@ Shortcut: ``E``
 A simple eraser tool. Left click erases single tiles and right click can
 be used to quickly erase rectangular areas.
 
+-  Holding ``Shift`` erases on all layers.
+
 Selection Tools
 ---------------
 

diff --git a/src/tiled/eraser.cpp b/src/tiled/eraser.cpp
@@ -38,7 +38,6 @@ Eraser::Eraser(QObject *parent)
                        QKeySequence(Qt::Key_E),
                        nullptr,
                        parent)
-    , mMode(Nothing)
 {
 }
 
@@ -59,7 +58,7 @@ void Eraser::mousePressed(QGraphicsSceneMouseEvent *event)
             mMode = Erase;
             doErase(false);
             return;
-        } else if (event->button() == Qt::RightButton && event->modifiers() == Qt::NoModifier) {
+        } else if (event->button() == Qt::RightButton) {
             mStart = tilePosition();
             mMode = RectangleErase;
             return;
@@ -88,6 +87,11 @@ void Eraser::mouseReleased(QGraphicsSceneMouseEvent *event)
     }
 }
 
+void Eraser::modifiersChanged(Qt::KeyboardModifiers modifiers)
+{
+    mAllLayers = modifiers & Qt::ShiftModifier;
+}
+
 void Eraser::languageChanged()
 {
     setName(tr("Eraser"));
@@ -105,17 +109,13 @@ void Eraser::doErase(bool continuation)
     }
     mLastTilePos = tilePos;
 
-    for (Layer *layer : mapDocument()->selectedLayers()) {
-        if (!layer->isTileLayer())
-            continue;
-        if (!layer->isUnlocked())
-            continue;
-
-        auto tileLayer = static_cast<TileLayer*>(layer);
+    auto eraseOnLayer = [&] (TileLayer *tileLayer) {
+        if (!tileLayer->isUnlocked())
+            return;
 
         QRegion eraseRegion = globalEraseRegion.intersected(tileLayer->bounds());
         if (eraseRegion.isEmpty())
-            continue;
+            return;
 
         EraseTiles *erase = new EraseTiles(mapDocument(), tileLayer, eraseRegion);
         erase->setMergeable(continuation);
@@ -124,6 +124,15 @@ void Eraser::doErase(bool continuation)
         emit mapDocument()->regionEdited(eraseRegion, tileLayer);
 
         continuation = true;    // further erases are always continuations
+    };
+
+    if (mAllLayers) {
+        for (Layer *layer : mapDocument()->map()->tileLayers())
+            eraseOnLayer(static_cast<TileLayer*>(layer));
+    } else {
+        for (Layer *layer : mapDocument()->selectedLayers())
+            if (TileLayer *tileLayer = layer->asTileLayer())
+                eraseOnLayer(tileLayer);
     }
 }
 

diff --git a/src/tiled/eraser.h b/src/tiled/eraser.h
@@ -37,6 +37,8 @@ class Eraser : public AbstractTileTool
     void mousePressed(QGraphicsSceneMouseEvent *event) override;
     void mouseReleased(QGraphicsSceneMouseEvent *event) override;
 
+    void modifiersChanged(Qt::KeyboardModifiers) override;
+
     void languageChanged() override;
 
 protected:
@@ -52,7 +54,8 @@ class Eraser : public AbstractTileTool
         RectangleErase
     };
 
-    Mode mMode;
+    Mode mMode = Nothing;
+    bool mAllLayers = false;
     QPoint mLastTilePos;
     QPoint mStart;
 };