panel_items : fixed change number of desktop

[chaz/tint2] / src / panel.c

diff --git a/src/panel.c b/src/panel.c
index a680bb480100af715dc061358784808a0658297b..2d34c573dc5a2558995d9c83d9e81ba018839358 100644 (file)
--- a/src/panel.c
+++ b/src/panel.c
@@ -80,7 +80,7 @@ void default_panel()
        task_dragged = 0;
        panel_horizontal = 1;
        panel_position = CENTER;
-       panel_items_order = strdup("LTBSC");  // Default order : Launcher, Taskbar, Battery, Systray, Clock
+       panel_items_order = 0;
        panel_autohide = 0;
        panel_autohide_show_timeout = 0;
        panel_autohide_hide_timeout = 0;
@@ -126,7 +126,7 @@ void cleanup_panel()
 
 void init_panel()
 {
-       int i, j;
+       int i, k;
        Panel *p;
 
        if (panel_config.monitor > (server.nb_monitor-1)) {
@@ -142,6 +142,7 @@ void init_panel()
 #ifdef ENABLE_BATTERY
        init_battery();
 #endif
+       init_taskbar();
 
        // number of panels (one monitor or 'all' monitors)
        if (panel_config.monitor >= 0)
@@ -172,38 +173,25 @@ void init_panel()
                p->g_taskbar.area.panel = p;
                p->g_task.area.panel = p;
                init_panel_size_and_position(p);
-               // add childs occording to panel_items_order
-               int k;
+               // add childs according to panel_items
                for (k=0 ; k < strlen(panel_items_order) ; k++) {
-                       if (panel_items_order[k] == 'L') {
+                       if (panel_items_order[k] == 'L') 
                                init_launcher_panel(p);
-                               p->area.list = g_slist_append(p->area.list, &p->launcher);
-                       }
-                       if (panel_items_order[k] == 'T') {
-                               p->nb_desktop = server.nb_desktop;
-                               p->taskbar = calloc(p->nb_desktop, sizeof(Taskbar));
-                               for (j=0 ; j < p->nb_desktop ; j++) {
-                                       p->area.list = g_slist_append(p->area.list, &p->taskbar[j]);
-                               }
-                               //printf("init taskbar\n");
-                       }
+                       if (panel_items_order[k] == 'T')
+                               init_taskbar_panel(p);
 #ifdef ENABLE_BATTERY
-                       if (panel_items_order[k] == 'B') {
+                       if (panel_items_order[k] == 'B')
                                init_battery_panel(p);
-                               p->area.list = g_slist_append(p->area.list, &p->battery);
-                       }
 #endif
                        if (panel_items_order[k] == 'S') {
                                // TODO : check systray is only on 1 panel
                                init_systray_panel(p);
-                               p->area.list = g_slist_append(p->area.list, &systray);
                                refresh_systray = 1;
                        }
-                       if (panel_items_order[k] == 'C') {
+                       if (panel_items_order[k] == 'C')
                                init_clock_panel(p);
-                               p->area.list = g_slist_append(p->area.list, &p->clock);
-                       }
                }
+               set_panel_items_order(p);
 
                // catch some events
                XSetWindowAttributes att = { .colormap=server.colormap, .background_pixel=0, .border_pixel=0 };
@@ -234,7 +222,6 @@ void init_panel()
        }
 
        panel_refresh = 1;
-       init_taskbar();
        visible_object();
        task_refresh_tasklist();
        active_task();
@@ -483,6 +470,36 @@ void update_strut(Panel* p)
 }
 
 
+void set_panel_items_order(Panel *p)
+{
+       int k, j;
+       
+       if (p->area.list) {
+               g_slist_free(p->area.list);
+               p->area.list = 0;
+       }
+
+       for (k=0 ; k < strlen(panel_items_order) ; k++) {
+               if (panel_items_order[k] == 'L') 
+                       p->area.list = g_slist_append(p->area.list, &p->launcher);
+               if (panel_items_order[k] == 'T') {
+                       for (j=0 ; j < p->nb_desktop ; j++)
+                               p->area.list = g_slist_append(p->area.list, &p->taskbar[j]);
+               }
+#ifdef ENABLE_BATTERY
+               if (panel_items_order[k] == 'B') 
+                       p->area.list = g_slist_append(p->area.list, &p->battery);
+#endif
+               if (panel_items_order[k] == 'S') {
+                       // TODO : check systray is only on 1 panel
+                       p->area.list = g_slist_append(p->area.list, &systray);
+               }
+               if (panel_items_order[k] == 'C')
+                       p->area.list = g_slist_append(p->area.list, &p->clock);
+       }
+}
+
+
 void set_panel_properties(Panel *p)
 {
        XStoreName (server.dsp, p->main_win, "tint2");