From 67c155c0ddff7d002eedfa2d35ee9203ccc276d3 Mon Sep 17 00:00:00 2001
From: Julien Danjou <julien@danjou.info>
Date: Thu, 17 Jan 2008 18:39:17 +0100
Subject: [PATCH] add tag_view() function

---
 tag.c | 44 +++++++++++++++++++++-----------------------
 tag.h |  1 +
 2 files changed, 22 insertions(+), 23 deletions(-)

diff --git a/tag.c b/tag.c
index a98b0da2..ca8f2332 100644
--- a/tag.c
+++ b/tag.c
@@ -286,15 +286,16 @@ uicb_tag_toggleview(int screen, char *arg)
 }
 
 static void
-tag_view_only(int screen, Tag *target)
+tag_view_only(Tag *target, int screen)
 {
     Tag *tag;
 
     if(!target) return;
 
     for(tag = globalconf.screens[screen].tags; tag; tag = tag->next)
-        tag->selected = False;
-    target->selected = True;
+        tag_view(tag, False, screen);
+
+    tag_view(target, True, screen);
 }
 
 void
@@ -307,7 +308,16 @@ tag_view_only_byindex(int screen, int dindex)
 
     for(tag = globalconf.screens[screen].tags; tag && dindex > 0;
         tag = tag->next, dindex--);
-    tag_view_only(screen, tag);
+    tag_view_only(tag, screen);
+}
+
+void
+tag_view(Tag *tag, Bool view, int screen)
+{
+    tag->selected = view;
+    ewmh_update_net_current_desktop(get_phys_screen(screen));
+    widget_invalidate_cache(screen, WIDGET_CACHE_TAGS);
+    saveawesomeprops(screen);
 }
 
 /** View tag
@@ -324,12 +334,9 @@ uicb_tag_view(int screen, char *arg)
 	tag_view_only_byindex(screen, atoi(arg) - 1);
     else
         for(tag = globalconf.screens[screen].tags; tag; tag = tag->next)
-            tag->selected = True;
+            tag_view(tag, True, screen);
 
-    saveawesomeprops(screen);
     arrange(screen);
-    ewmh_update_net_current_desktop(get_phys_screen(screen));
-    widget_invalidate_cache(screen, WIDGET_CACHE_TAGS);
 }
 
 /** View previously selected tags
@@ -346,13 +353,11 @@ uicb_tag_prev_selected(int screen, char *arg __attribute__ ((unused)))
     for(tag =  globalconf.screens[screen].tags; tag; tag = tag->next)
     {
         t = tag->selected;
-        tag->selected = tag->was_selected;
+        tag_view(tag, tag->was_selected, screen);
         tag->was_selected = t;
     }
-    saveawesomeprops(screen);
+
     arrange(screen);
-    ewmh_update_net_current_desktop(get_phys_screen(screen));
-    widget_invalidate_cache(screen, WIDGET_CACHE_TAGS);
 }
 
 /** View next tag
@@ -370,15 +375,11 @@ uicb_tag_viewnext(int screen, char *arg __attribute__ ((unused)))
     else
         tag = globalconf.screens[screen].tags;
 
-    curtags[0]->selected = False;
-    tag->selected = True;
+    tag_view(curtags[0], False, screen);
+    tag_view(tag, True, screen);
 
     p_delete(&curtags);
-
-    saveawesomeprops(screen);
     arrange(screen);
-    ewmh_update_net_current_desktop(get_phys_screen(screen));
-    widget_invalidate_cache(screen, WIDGET_CACHE_TAGS);
 }
 
 /** View previous tag
@@ -393,14 +394,11 @@ uicb_tag_viewprev(int screen, char *arg __attribute__ ((unused)))
 
     tag = tag_list_prev_cycle(&globalconf.screens[screen].tags, curtags[0]);
 
-    curtags[0]->selected = False;
-    tag->selected = True;
+    tag_view(curtags[0], False, screen);
+    tag_view(tag, True, screen);
 
     p_delete(&curtags);
-    saveawesomeprops(screen);
     arrange(screen);
-    ewmh_update_net_current_desktop(get_phys_screen(screen));
-    widget_invalidate_cache(screen, WIDGET_CACHE_TAGS);
 }
 
 void
diff --git a/tag.h b/tag.h
index 57d605ba..d1e7d690 100644
--- a/tag.h
+++ b/tag.h
@@ -28,6 +28,7 @@
 #define IS_TILED(client, screen)            (client && !client->isfloating && client_isvisible(client, screen))
 
 Tag * tag_new(const char *, Layout *, double, int, int);
+void tag_view(Tag *, Bool, int);
 void tag_push_to_screen(Tag *, int);
 Tag ** get_current_tags(int);
 void tag_client(Client *, Tag *);