[cheese] Fix de-activation of effects button



commit ac1852761a30449fb1eb96325800cab34cd6df6c
Author: Hans de Goede <hdegoede redhat com>
Date:   Tue Jun 11 12:51:09 2013 +0200

    Fix de-activation of effects button
    
    So that the user does not need to click twice on the effect button to change
    the effect (after the first time the effect was changed).
    
    https://bugzilla.gnome.org/show_bug.cgi?id=698399
    
    Signed-off-by: Hans de Goede <hdegoede redhat com>

 src/cheese-window.vala |    9 +++------
 1 files changed, 3 insertions(+), 6 deletions(-)
---
diff --git a/src/cheese-window.vala b/src/cheese-window.vala
index fdca413..e8994e9 100644
--- a/src/cheese-window.vala
+++ b/src/cheese-window.vala
@@ -811,10 +811,9 @@ public class Cheese.MainWindow : Gtk.ApplicationWindow
         }
         action_cancelled = false;
       }
-      else
-      if (is_effects_selector_active)
+      else if (is_effects_selector_active)
       {
-        // FIXME: Set the effects action to be inactive.
+        effects_toggle_button.set_active (false);
       }
     }
     return false;
@@ -950,7 +949,6 @@ public class Cheese.MainWindow : Gtk.ApplicationWindow
     public void set_effects (bool effects)
     {
         toggle_effects_selector (effects);
-        // FIXME: Set the mode action to be inverse sensitivity to effects.
     }
 
   /**
@@ -964,12 +962,11 @@ public class Cheese.MainWindow : Gtk.ApplicationWindow
                                          Clutter.ButtonEvent event)
   {
     /* Disable the effects selector after selecting an effect. */
-    toggle_effects_selector(false);
+    effects_toggle_button.set_active (false);
 
     selected_effect = source.get_data ("effect");
     camera.set_effect (selected_effect);
     settings.set_string ("selected-effect", selected_effect.name);
-    // FIXME: Set the effects action to be inactive.
     return false;
   }
 


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]