[gnome-music/wip/mschraal/window-template: 4/4] playertoolbar: Simplify repeat mode icon code



commit b5fa7daa3e4d775db9b19940316e5163528bf91c
Author: Marinus Schraal <mschraal gnome org>
Date:   Mon Sep 10 02:27:52 2018 +0200

    playertoolbar: Simplify repeat mode icon code

 gnomemusic/widgets/playertoolbar.py | 20 ++++++++------------
 1 file changed, 8 insertions(+), 12 deletions(-)
---
diff --git a/gnomemusic/widgets/playertoolbar.py b/gnomemusic/widgets/playertoolbar.py
index 89209588..2ad60b16 100644
--- a/gnomemusic/widgets/playertoolbar.py
+++ b/gnomemusic/widgets/playertoolbar.py
@@ -62,6 +62,13 @@ class PlayerToolbar(Gtk.ActionBar):
     _song_info_box = Gtk.Template.Child()
     _title_label = Gtk.Template.Child()
 
+    _repeat_dict = {
+        RepeatMode.ALL: 'media-playlist-repeat-symbolic',
+        RepeatMode.NONE: 'media-playlist-consecutive-symbolic',
+        RepeatMode.SHUFFLE: 'media-playlist-shuffle-symbolic',
+        RepeatMode.SONG: 'media-playlist-repeat-song-symbolic'
+    }
+
     def __repr__(self):
         return '<PlayerToolbar>'
 
@@ -138,19 +145,8 @@ class PlayerToolbar(Gtk.ActionBar):
         self._sync_repeat_image()
         self._sync_prev_next()
 
-    @log
     def _sync_repeat_image(self):
-        icon = None
-        repeat_mode = self._player.props.repeat_mode
-        if repeat_mode == RepeatMode.NONE:
-            icon = 'media-playlist-consecutive-symbolic'
-        elif repeat_mode == RepeatMode.SHUFFLE:
-            icon = 'media-playlist-shuffle-symbolic'
-        elif repeat_mode == RepeatMode.ALL:
-            icon = 'media-playlist-repeat-symbolic'
-        elif repeat_mode == RepeatMode.SONG:
-            icon = 'media-playlist-repeat-song-symbolic'
-
+        icon = self._repeat_dict[self._player.props.repeat_mode]
         self._repeat_image.set_from_icon_name(icon, Gtk.IconSize.MENU)
 
     @log


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