From bbc61a321555a5c3602547829341c7d88e70ab54 Mon Sep 17 00:00:00 2001
From: Sophie Herold <sophie@hemio.de>
Date: Wed, 15 Jan 2025 22:10:30 +0100
Subject: [PATCH] edit_window: Disable "Done" if image is unchanged

---
 src/widgets/edit_window.rs | 12 +++++++++++-
 src/widgets/edit_window.ui | 11 ++++++-----
 2 files changed, 17 insertions(+), 6 deletions(-)

diff --git a/src/widgets/edit_window.rs b/src/widgets/edit_window.rs
index 7a0b7af..9577a7f 100644
--- a/src/widgets/edit_window.rs
+++ b/src/widgets/edit_window.rs
@@ -41,7 +41,7 @@ mod imp {
         #[template_child]
         cancel: TemplateChild<gtk::Button>,
         #[template_child]
-        done: TemplateChild<gtk::MenuButton>,
+        pub(crate) done: TemplateChild<gtk::MenuButton>,
 
         #[property(get, construct_only)]
         original_image: OnceCell<LpImage>,
@@ -263,6 +263,16 @@ impl LpEditWindow {
     pub fn set_operations(&self, operations: Option<Arc<glycin::Operations>>) {
         let imp = self.imp();
 
+        let done_sensitive = operations
+            .as_ref()
+            .is_some_and(|x| !x.operations().is_empty());
+        if done_sensitive {
+            imp.done.add_css_class("suggested-action");
+        } else {
+            imp.done.remove_css_class("suggested-action");
+        }
+        imp.done.set_sensitive(done_sensitive);
+
         imp.operations.replace(operations);
     }
 
diff --git a/src/widgets/edit_window.ui b/src/widgets/edit_window.ui
index 5991766..1559eb8 100644
--- a/src/widgets/edit_window.ui
+++ b/src/widgets/edit_window.ui
@@ -19,6 +19,10 @@
               <object class="GtkMenuButton" id="done">
                 <property name="label" translatable="true">_Done</property>
                 <property name="use-underline">true</property>
+                <property name="sensitive">false</property>
+                <style>
+                  <class name="raised" />
+                </style>
                 <property name="popover">
                   <object class="GtkPopover">
                     <style>
@@ -46,7 +50,7 @@
                                     <property name="label" translatable="yes">Keep original and create a new file</property>
                                     <property name="xalign">0</property>
                                     <style>
-                                      <class name="dim-label" />
+                                      <class name="dimmed" />
                                       <class name="caption" />
                                     </style>
                                   </object>
@@ -74,7 +78,7 @@
                                     <property name="label" translatable="yes">Original image will be moved to the trash</property>
                                     <property name="xalign">0</property>
                                     <style>
-                                      <class name="dim-label" />
+                                      <class name="dimmed" />
                                       <class name="caption" />
                                     </style>
                                   </object>
@@ -87,9 +91,6 @@
                     </property>
                   </object>
                 </property>
-                <style>
-                  <class name="suggested-action" />
-                </style>
               </object>
             </child>
           </object>