From 46214456f14c91231a9726272a01c4972111d9d7 Mon Sep 17 00:00:00 2001
From: Lonami Exo <totufals@hotmail.com>
Date: Tue, 21 Feb 2017 21:11:04 +0100
Subject: [PATCH] Tiny improvement to toggle settings

---
 core/src/io/github/lonamiwebs/klooni/Klooni.java     | 12 ++++++++----
 .../lonamiwebs/klooni/screens/CustomizeScreen.java   |  8 ++++----
 2 files changed, 12 insertions(+), 8 deletions(-)

diff --git a/core/src/io/github/lonamiwebs/klooni/Klooni.java b/core/src/io/github/lonamiwebs/klooni/Klooni.java
index 8c39939..fecdca1 100644
--- a/core/src/io/github/lonamiwebs/klooni/Klooni.java
+++ b/core/src/io/github/lonamiwebs/klooni/Klooni.java
@@ -119,16 +119,20 @@ public class Klooni extends Game {
         return !prefs.getBoolean("muteSound", false);
     }
 
-    public static void toggleSound() {
-        prefs.putBoolean("muteSound", soundsEnabled()).flush();
+    public static boolean toggleSound() {
+        final boolean result = soundsEnabled();
+        prefs.putBoolean("muteSound", result).flush();
+        return !result;
     }
 
     public static boolean shouldSnapToGrid() {
         return prefs.getBoolean("snapToGrid", false);
     }
 
-    public static void toggleSnapToGrid() {
-        prefs.putBoolean("snapToGrid", !shouldSnapToGrid()).flush();
+    public static boolean toggleSnapToGrid() {
+        final boolean result = !shouldSnapToGrid();
+        prefs.putBoolean("snapToGrid", result).flush();
+        return result;
     }
 
     public static boolean isThemeBought(Theme theme) {
diff --git a/core/src/io/github/lonamiwebs/klooni/screens/CustomizeScreen.java b/core/src/io/github/lonamiwebs/klooni/screens/CustomizeScreen.java
index cccbee0..8ff0549 100644
--- a/core/src/io/github/lonamiwebs/klooni/screens/CustomizeScreen.java
+++ b/core/src/io/github/lonamiwebs/klooni/screens/CustomizeScreen.java
@@ -73,9 +73,9 @@ class CustomizeScreen implements Screen {
         soundButton.addListener(new ChangeListener() {
             @Override
             public void changed(ChangeEvent event, Actor actor) {
-                Klooni.toggleSound();
+                final boolean enabled = Klooni.toggleSound();
                 soundButton.image = CustomizeScreen.this.game.skin.getDrawable(
-                        Klooni.soundsEnabled() ? "sound_on_texture" : "sound_off_texture");
+                        enabled ? "sound_on_texture" : "sound_off_texture");
             }
         });
         optionsGroup.addActor(soundButton);
@@ -87,9 +87,9 @@ class CustomizeScreen implements Screen {
         snapButton.addListener(new ChangeListener() {
             @Override
             public void changed(ChangeEvent event, Actor actor) {
-                Klooni.toggleSnapToGrid();
+                final boolean shouldSnap = Klooni.toggleSnapToGrid();
                 snapButton.image = CustomizeScreen.this.game.skin.getDrawable(
-                        Klooni.shouldSnapToGrid() ? "snap_on_texture" : "snap_off_texture");
+                        shouldSnap ? "snap_on_texture" : "snap_off_texture");
             }
         });
         optionsGroup.addActor(snapButton);