[gnome-games/wip/exalm/views: 10/14] application-window: Replace 'ui-state' with 'current-view'

[Alexander Mikhaylenko <alexm src gnome org>]

commit ab368021bb81a495deacdf396b86d7fc2a180d15
Author: Alexander Mikhaylenko <exalm7659 gmail com>
Date:   Thu Oct 4 20:44:10 2018 +0500

    application-window: Replace 'ui-state' with 'current-view'

 src/ui/application-window.vala | 88 ++++++++++++++++--------------------------
 src/ui/display-view.vala       |  8 +++-
 2 files changed, 40 insertions(+), 56 deletions(-)
---
diff --git a/src/ui/application-window.vala b/src/ui/application-window.vala
index c6d8178b..d6b22d82 100644
--- a/src/ui/application-window.vala
+++ b/src/ui/application-window.vala
@@ -7,38 +7,23 @@ private class Games.ApplicationWindow : Gtk.ApplicationWindow {
 
        private const string CONTRIBUTE_URI = "https://wiki.gnome.org/Apps/Games/Contribute";;
 
-       private UiState _ui_state;
-       public UiState ui_state {
-               get { return _ui_state; }
+       private ApplicationView _current_view;
+       public ApplicationView current_view {
+               get { return _current_view; }
                set {
-                       if (value == ui_state)
+                       if (value == current_view)
                                return;
 
-                       _ui_state = value;
+                       if (current_view != null)
+                               current_view.is_view_active = false;
 
-                       switch (ui_state) {
-                       case UiState.COLLECTION:
-                               content_box.visible_child = collection_view;
-                               header_bar.visible_child = collection_view.header_bar;
+                       _current_view = value;
 
-                               display_view.is_view_active = false;
-                               collection_view.is_view_active = true;
+                       content_box.visible_child = current_view;
+                       header_bar.visible_child = current_view.header_bar;
 
-                               if (display_view.display_box.runner != null) {
-                                       display_view.display_box.runner.stop ();
-                                       display_view.display_box.runner = null;
-                               }
-
-                               break;
-                       case UiState.DISPLAY:
-                               content_box.visible_child = display_view;
-                               header_bar.visible_child = display_view.header_bar;
-
-                               collection_view.is_view_active = false;
-                               display_view.is_view_active = true;
-
-                               break;
-                       }
+                       if (current_view != null)
+                               current_view.is_view_active = true;
 
                        konami_code.reset ();
                }
@@ -48,7 +33,7 @@ private class Games.ApplicationWindow : Gtk.ApplicationWindow {
        public bool is_fullscreen {
                get { return _is_fullscreen; }
                set {
-                       _is_fullscreen = value && (ui_state == UiState.DISPLAY);
+                       _is_fullscreen = value && (current_view == display_view);
 
                        if (_is_fullscreen)
                                fullscreen ();
@@ -134,8 +119,6 @@ private class Games.ApplicationWindow : Gtk.ApplicationWindow {
 
                if (Config.PROFILE == "Devel")
                        get_style_context ().add_class ("devel");
-
-               ui_state = UiState.COLLECTION;
        }
 
        public void run_game (Game game) {
@@ -206,8 +189,8 @@ private class Games.ApplicationWindow : Gtk.ApplicationWindow {
                        return true;
                }
 
-               if (ui_state == UiState.COLLECTION)
-                       return collection_view.on_key_pressed (event);
+               if (current_view == collection_view)
+                       return current_view.on_key_pressed (event);
 
                return handle_display_key_event (event);
        }
@@ -216,7 +199,7 @@ private class Games.ApplicationWindow : Gtk.ApplicationWindow {
        public bool on_button_pressed (Gdk.EventButton event) {
                // Mouse button 8 is the navigation previous button
                if (event.button == 8) {
-                       if (ui_state != UiState.DISPLAY)
+                       if (current_view != display_view)
                                return false;
 
                        on_display_back ();
@@ -238,7 +221,7 @@ private class Games.ApplicationWindow : Gtk.ApplicationWindow {
                        return false;
 
                var focused = (bool) (event.new_window_state & Gdk.WindowState.FOCUSED);
-               var playing = (ui_state == UiState.DISPLAY);
+               var playing = (current_view == display_view);
 
                if (focused && playing)
                        inhibit (Gtk.ApplicationInhibitFlags.IDLE);
@@ -250,10 +233,9 @@ private class Games.ApplicationWindow : Gtk.ApplicationWindow {
        }
 
        public bool gamepad_button_press_event (Manette.Event event) {
-               switch (ui_state) {
-               case UiState.COLLECTION:
+               if (current_view == collection_view)
                        return is_active && collection_view.collection_box.gamepad_button_press_event (event);
-               case UiState.DISPLAY:
+               else if (current_view == display_view) {
                        if (resume_dialog != null)
                                return resume_dialog.is_active && resume_dialog.gamepad_button_press_event 
(event);
 
@@ -272,33 +254,29 @@ private class Games.ApplicationWindow : Gtk.ApplicationWindow {
 
                        switch (button) {
                        case EventCode.BTN_MODE:
-                               ui_state = UiState.COLLECTION;
+                               current_view = collection_view;
 
                                return true;
                        default:
                                return false;
                        }
-               default:
-                       return false;
                }
+
+               return false;
        }
 
        public bool gamepad_button_release_event (Manette.Event event) {
-               switch (ui_state) {
-               case UiState.COLLECTION:
+               if (current_view == collection_view)
                        return is_active && collection_view.collection_box.gamepad_button_release_event 
(event);
-               default:
-                       return false;
-               }
+
+               return false;
        }
 
        public bool gamepad_absolute_axis_event (Manette.Event event) {
-               switch (ui_state) {
-               case UiState.COLLECTION:
+               if (current_view == collection_view)
                        return is_active && collection_view.collection_box.gamepad_absolute_axis_event 
(event);
-               default:
-                       return false;
-               }
+
+               return false;
        }
 
        [GtkCallback]
@@ -309,7 +287,7 @@ private class Games.ApplicationWindow : Gtk.ApplicationWindow {
        [GtkCallback]
        private void on_display_back () {
                if (quit_game ())
-                       ui_state = UiState.COLLECTION;
+                       current_view = collection_view;
 
                uninhibit (Gtk.ApplicationInhibitFlags.IDLE | Gtk.ApplicationInhibitFlags.LOGOUT);
        }
@@ -317,7 +295,7 @@ private class Games.ApplicationWindow : Gtk.ApplicationWindow {
        private void run_game_with_cancellable (Game game, Cancellable cancellable) {
                display_view.display_header_bar.game_title = game.name;
                display_view.display_box.header_bar.game_title = game.name;
-               ui_state = UiState.DISPLAY;
+               current_view = display_view;
 
                // Reset the UI parts depending on the runner to avoid an
                // inconsistent state is case we couldn't retrieve it.
@@ -426,7 +404,7 @@ private class Games.ApplicationWindow : Gtk.ApplicationWindow {
 
                if (response == Gtk.ResponseType.CANCEL) {
                        display_view.display_box.runner = null;
-                       ui_state = UiState.COLLECTION;
+                       current_view = collection_view;
 
                        return;
                }
@@ -561,7 +539,7 @@ private class Games.ApplicationWindow : Gtk.ApplicationWindow {
        }
 
        private bool can_update_pause () {
-               if (ui_state != UiState.DISPLAY)
+               if (current_view != display_view)
                        return false;
 
                if (display_view.display_box.runner == null)
@@ -577,7 +555,7 @@ private class Games.ApplicationWindow : Gtk.ApplicationWindow {
        }
 
        private bool handle_display_key_event (Gdk.EventKey event) {
-               if (ui_state != UiState.DISPLAY)
+               if (current_view != display_view)
                        return false;
 
                var default_modifiers = Gtk.accelerator_get_default_mod_mask ();
@@ -656,7 +634,7 @@ private class Games.ApplicationWindow : Gtk.ApplicationWindow {
        }
 
        private void on_konami_code_performed () {
-               if (ui_state != UiState.COLLECTION)
+               if (current_view != collection_view)
                        return;
 
                try {
diff --git a/src/ui/display-view.vala b/src/ui/display-view.vala
index 7da8dbd8..aef7e197 100644
--- a/src/ui/display-view.vala
+++ b/src/ui/display-view.vala
@@ -23,8 +23,14 @@ private class Games.DisplayView: Gtk.Bin, ApplicationView {
 
                        _is_view_active = value;
 
-                       if (!is_view_active)
+                       if (!is_view_active) {
                                is_fullscreen = false;
+
+                               if (display_box.runner != null) {
+                                       display_box.runner.stop ();
+                                       display_box.runner = null;
+                               }
+                       }
                }
        }