From 129d1cfc7a9445d2368f809176a245f2e9571757 Mon Sep 17 00:00:00 2001
From: Sophie Herold <sophie@hemio.de>
Date: Wed, 29 Jan 2025 12:29:26 +0100
Subject: [PATCH] edit/crop: Add keyboard shortcuts

Closes #442
---
 data/resources/gtk/help-overlay.ui | 35 ++++++++++++++++++++
 src/widgets/edit/crop.rs           | 52 ++++++++++++++++++++++++++++++
 src/widgets/edit/crop_selection.rs |  2 ++
 3 files changed, 89 insertions(+)

diff --git a/data/resources/gtk/help-overlay.ui b/data/resources/gtk/help-overlay.ui
index 42951d2..f5f9ce6 100644
--- a/data/resources/gtk/help-overlay.ui
+++ b/data/resources/gtk/help-overlay.ui
@@ -283,6 +283,41 @@
             </child>
           </object>
         </child>
+        <child>
+          <object class="GtkShortcutsGroup">
+            <property name="title" translatable="yes" context="shortcut window">Editing</property>
+            <child>
+              <object class="GtkShortcutsShortcut">
+                <property name="title" translatable="yes" context="shortcut window">Rotate Clockwise</property>
+                <property name="accelerator">&lt;Ctrl&gt;R</property>
+              </object>
+            </child>
+            <child>
+              <object class="GtkShortcutsShortcut">
+                <property name="title" translatable="yes" context="shortcut window">Rotate Counterclockwise</property>
+                <property name="accelerator">&lt;Ctrl&gt;&lt;Shift&gt;R</property>
+              </object>
+            </child>
+            <child>
+              <object class="GtkShortcutsShortcut">
+                <property name="title" translatable="yes" context="shortcut window">Mirror Horizontally</property>
+                <property name="accelerator">H</property>
+              </object>
+            </child>
+            <child>
+              <object class="GtkShortcutsShortcut">
+                <property name="title" translatable="yes" context="shortcut window">Mirror Vertically</property>
+                <property name="accelerator">V</property>
+              </object>
+            </child>
+            <child>
+              <object class="GtkShortcutsShortcut">
+                <property name="title" translatable="yes" context="shortcut window">Apply Crop</property>
+                <property name="accelerator">&lt;Ctrl&gt;Return</property>
+              </object>
+            </child>
+          </object>
+        </child>
       </object>
     </child>
   </object>
diff --git a/src/widgets/edit/crop.rs b/src/widgets/edit/crop.rs
index 1a9f609..8bb7928 100644
--- a/src/widgets/edit/crop.rs
+++ b/src/widgets/edit/crop.rs
@@ -60,6 +60,8 @@ pub enum LpOrientation {
 
 mod imp {
 
+    use gtk::gdk;
+
     use super::*;
 
     #[derive(Debug, Default, gtk::CompositeTemplate, glib::Properties)]
@@ -105,6 +107,56 @@ mod imp {
             klass.install_action("edit-crop.apply-crop", None, |obj, _, _| {
                 obj.imp().apply_crop()
             });
+
+            klass.add_shortcut(
+                &gtk::Shortcut::builder()
+                    .action(&gtk::NamedAction::new("edit-crop.mirror-horizontally"))
+                    .trigger(&gtk::KeyvalTrigger::new(
+                        gdk::Key::H,
+                        gdk::ModifierType::NO_MODIFIER_MASK,
+                    ))
+                    .build(),
+            );
+
+            klass.add_shortcut(
+                &gtk::Shortcut::builder()
+                    .action(&gtk::NamedAction::new("edit-crop.mirror-vertically"))
+                    .trigger(&gtk::KeyvalTrigger::new(
+                        gdk::Key::V,
+                        gdk::ModifierType::NO_MODIFIER_MASK,
+                    ))
+                    .build(),
+            );
+
+            klass.add_shortcut(
+                &gtk::Shortcut::builder()
+                    .action(&gtk::NamedAction::new("edit-crop.rotate-cw"))
+                    .trigger(&gtk::KeyvalTrigger::new(
+                        gdk::Key::R,
+                        gdk::ModifierType::CONTROL_MASK,
+                    ))
+                    .build(),
+            );
+
+            klass.add_shortcut(
+                &gtk::Shortcut::builder()
+                    .action(&gtk::NamedAction::new("edit-crop.rotate-ccw"))
+                    .trigger(&gtk::KeyvalTrigger::new(
+                        gdk::Key::R,
+                        gdk::ModifierType::CONTROL_MASK.union(gdk::ModifierType::SHIFT_MASK),
+                    ))
+                    .build(),
+            );
+
+            klass.add_shortcut(
+                &gtk::Shortcut::builder()
+                    .action(&gtk::NamedAction::new("edit-crop.apply-crop"))
+                    .trigger(&gtk::KeyvalTrigger::new(
+                        gdk::Key::Return,
+                        gdk::ModifierType::CONTROL_MASK,
+                    ))
+                    .build(),
+            );
         }
 
         fn instance_init(obj: &glib::subclass::InitializingObject<Self>) {
diff --git a/src/widgets/edit/crop_selection.rs b/src/widgets/edit/crop_selection.rs
index e68169e..27db2a1 100644
--- a/src/widgets/edit/crop_selection.rs
+++ b/src/widgets/edit/crop_selection.rs
@@ -194,6 +194,8 @@ mod imp {
 
         #[template_child]
         selection_move: TemplateChild<gtk::GestureDrag>,
+        #[template_child]
+        shortcuts: TemplateChild<gtk::ShortcutController>,
 
         #[template_child]
         selection_overlay: TemplateChild<gtk::Overlay>,