"Search actions" widget

NEWS.md

@@ -1,5 +1,6 @@
 ### Unreleased
 
+* Added action search popup on Ctrl+Shift+P (with dogboydog, #3449)
 * Fixed new layer names to be always unique (by Logan Higinbotham, #3452)
 * Scripting: Added Object.setColorProperty and Object.setFloatProperty (#3423)
 * Scripting: Added tiled.projectFilePath

docs/conf.py

@@ -67,7 +67,7 @@
 #
 # This is also used if you do content translation via gettext catalogs.
 # Usually you set "language" from the command line for these cases.
-language = None
+language = 'en'
 
 # Options for localization
 locale_dirs = ['locale/']

docs/manual/introduction.rst

@@ -63,15 +63,22 @@ as well as to be able to quickly switch between multiple projects, it is
 recommended to first set up a :doc:`Tiled project <projects>`. This is however
 an entirely optional step that can be skipped when desired.
 
-Choose *Project -> Save Project As...* to save a new project file. The
-recommended location is the root of your project, but you can place it
-anywhere you want.
-
-Next, we'll add at least one folder, either some "assets" folder or simply the
-root of your project, but you can also choose to add several top-level folders
-like "tilesets", "maps", "templates", etc. Right-click in the Project view and
-choose *Add Folder to Project...* to add the relevant folders.
-
+Choose *File -> New -> New Project...* to create a new project file. It is
+recommended to save this file in the root of your project. The directory in
+which you store the project will be automatically added, so that its files are
+visible in the Project view.
+
+When necessary, you can add additional folders to the project or replace the
+one added by default. For example, you could choose to add several top-level
+folders like "tilesets", "maps", "templates", etc. Right-click in the
+Project view and choose *Add Folder to Project...* to add the
+relevant folders.
+
+.. hint::
+
+   You can press ``Ctrl+Shift+P`` to open the action search widget,
+   which can provide a faster way to get to actions than looking for them in
+   the menus!
 
 Creating a New Map
 ~~~~~~~~~~~~~~~~~~

docs/manual/keyboard-shortcuts.rst

@@ -14,6 +14,7 @@ General
 -  ``Ctrl + N`` - Create a new map
 -  ``Ctrl + O`` - Open any file or project
 -  ``Ctrl + P`` - Open a file in the current project
+-  ``Ctrl + Shift + P`` - Search for available actions
 -  ``Ctrl + Shift + T`` - Reopen a recently closed file
 -  ``Ctrl + S`` - Save current document
 -  ``Ctrl + Alt + S`` - Save current document to another file

src/tiled/abstractobjecttool.cpp

@@ -130,9 +130,23 @@ AbstractObjectTool::AbstractObjectTool(Id id,
     connect(mRotateLeft, &QAction::triggered, this, &AbstractObjectTool::rotateLeft);
     connect(mRotateRight, &QAction::triggered, this, &AbstractObjectTool::rotateRight);
 
+    setActionsEnabled(false);
+
     AbstractObjectTool::languageChanged();
 }
 
+void AbstractObjectTool::activate(MapScene *scene)
+{
+    AbstractTool::activate(scene);
+    setActionsEnabled(true);
+}
+
+void AbstractObjectTool::deactivate(MapScene *scene)
+{
+    setActionsEnabled(false);
+    AbstractTool::deactivate(scene);
+}
+
 void AbstractObjectTool::keyPressed(QKeyEvent *event)
 {
     switch (event->key()) {
@@ -739,4 +753,12 @@ void AbstractObjectTool::showContextMenu(MapObject *clickedObject,
     }
 }
 
+void AbstractObjectTool::setActionsEnabled(bool enabled)
+{
+    mFlipHorizontal->setEnabled(enabled);
+    mFlipVertical->setEnabled(enabled);
+    mRotateLeft->setEnabled(enabled);
+    mRotateRight->setEnabled(enabled);
+}
+
 #include "moc_abstractobjecttool.cpp"

src/tiled/abstractobjecttool.h

@@ -59,6 +59,9 @@ class AbstractObjectTool : public AbstractTool
                        const QKeySequence &shortcut,
                        QObject *parent = nullptr);
 
+    void activate(MapScene *scene) override;
+    void deactivate(MapScene *scene) override;
+
     void keyPressed(QKeyEvent *event) override;
     void mouseLeft() override;
     void mouseMoved(const QPointF &pos, Qt::KeyboardModifiers modifiers) override;
@@ -107,6 +110,8 @@ class AbstractObjectTool : public AbstractTool
     void showContextMenu(MapObject *clickedObject,
                          QPoint screenPos);
 
+    void setActionsEnabled(bool enabled);
+
     QAction *mFlipHorizontal;
     QAction *mFlipVertical;
     QAction *mRotateLeft;

src/tiled/abstracttilefilltool.cpp

@@ -60,9 +60,16 @@ AbstractTileFillTool::~AbstractTileFillTool()
 {
 }
 
+void AbstractTileFillTool::activate(MapScene *scene)
+{
+    AbstractTileTool::activate(scene);
+    mStampActions->setEnabled(true);
+}
+
 void AbstractTileFillTool::deactivate(MapScene *scene)
 {
     mCaptureStampHelper.reset();
+    mStampActions->setEnabled(false);
     AbstractTileTool::deactivate(scene);
 }
 

src/tiled/abstracttilefilltool.h

@@ -53,6 +53,7 @@ class AbstractTileFillTool : public AbstractTileTool
                          QObject *parent = nullptr);
     ~AbstractTileFillTool() override;
 
+    void activate(MapScene *scene) override;
     void deactivate(MapScene *scene) override;
 
     void mousePressed(QGraphicsSceneMouseEvent *event) override;

src/tiled/actionmanager.cpp

@@ -161,6 +161,19 @@ QAction *ActionManager::findAction(Id id)
     return d->mIdToActions.value(id);
 }
 
+QAction *ActionManager::findEnabledAction(Id id)
+{
+    auto d = instance();
+
+    const auto [start, end] = qAsConst(d->mIdToActions).equal_range(id);
+    for (auto it = start; it != end; ++it) {
+        if (it.value()->isEnabled())
+            return it.value();
+    }
+
+    return nullptr;
+}
+
 bool ActionManager::hasMenu(Id id)
 {
     return instance()->mIdToMenu.contains(id);

src/tiled/actionmanager.h

@@ -80,6 +80,7 @@ class ActionManager : public QObject
 
     static QAction *action(Id id);
     static QAction *findAction(Id id);
+    static QAction *findEnabledAction(Id id);
 
     static bool hasMenu(Id id);