From 1537ad2673843e033035edfe144ffc78bdab5a43 Mon Sep 17 00:00:00 2001 From: Hans de Goede Date: Tue, 6 Aug 2013 14:50:00 +0200 Subject: [PATCH 03/13] cheese-window: Allow changing effects while shooting / recording a video There is no technical reason to disable the effect selector while shooting a video. Likewise there is no technical reason to disallow start / stop recording / shooting, or changing mode when the effect selector is change. Removing these unnecessary limitations simplifies the code, and allows users to change effects while recording a video. Signed-off-by: Hans de Goede --- src/cheese-application.vala | 25 +++---------------------- src/cheese-window.vala | 4 ---- 2 files changed, 3 insertions(+), 26 deletions(-) diff --git a/src/cheese-application.vala b/src/cheese-application.vala index 0f9f33d..9cb5ac5 100644 --- a/src/cheese-application.vala +++ b/src/cheese-application.vala @@ -368,17 +368,8 @@ public class Cheese.Application : Gtk.Application switch (new_state) { case Gst.State.PLAYING: - if (effects.state.get_boolean ()) - { - mode.set_enabled (false); - shoot.set_enabled (false); - } - else - { - mode.set_enabled (true); - shoot.set_enabled (true); - } - + mode.set_enabled (true); + shoot.set_enabled (true); effects.set_enabled (true); main_window.camera_state_change_playing (); @@ -491,17 +482,7 @@ public class Cheese.Application : Gtk.Application { return_if_fail (value != null); - var state = value.get_boolean (); - - var shoot = this.lookup_action ("shoot") as SimpleAction; - var mode = this.lookup_action ("mode") as SimpleAction; - - // Effects selection and shooting/mode changes are mutually exclusive. - shoot.set_enabled (!state); - mode.set_enabled (!state); - - main_window.set_effects (state); - + main_window.set_effects (value.get_boolean ()); action.set_state (value); } diff --git a/src/cheese-window.vala b/src/cheese-window.vala index 675a92e..07cf0f4 100644 --- a/src/cheese-window.vala +++ b/src/cheese-window.vala @@ -384,8 +384,6 @@ public class Cheese.MainWindow : Gtk.ApplicationWindow { var mode = this.application.lookup_action ("mode") as SimpleAction; mode.set_enabled (true); - var effects = this.application.lookup_action ("effects") as SimpleAction; - effects.set_enabled (true); mode_change_changed (true); } @@ -396,8 +394,6 @@ public class Cheese.MainWindow : Gtk.ApplicationWindow { var mode = this.application.lookup_action ("mode") as SimpleAction; mode.set_enabled (false); - var effects = this.application.lookup_action ("effects") as SimpleAction; - effects.set_enabled (false); mode_change_changed (false); } -- 1.8.3.1