From 7bac08ef52118d89634bef6488b49dbb869dc253 Mon Sep 17 00:00:00 2001
From: markt <markt>
Date: Sun, 7 Jan 2007 18:45:30 +0000
Subject: another little bug with iconbar rendering

---
 src/IconbarTool.cc | 18 ++++++++++++++----
 1 file changed, 14 insertions(+), 4 deletions(-)

diff --git a/src/IconbarTool.cc b/src/IconbarTool.cc
index ad4eeeb..30db541 100644
--- a/src/IconbarTool.cc
+++ b/src/IconbarTool.cc
@@ -430,6 +430,7 @@ void IconbarTool::hide() {
 void IconbarTool::setAlignment(Container::Alignment align) {
     *m_rc_alignment = align;
     update(0);
+    m_menu.reconfigure();
 }
 
 void IconbarTool::setMode(Mode mode) {
@@ -453,6 +454,8 @@ void IconbarTool::setMode(Mode mode) {
     m_icon_container.showSubwindows();
 
     renderTheme();
+
+    m_menu.reconfigure();
 }
 
 unsigned int IconbarTool::width() const {
@@ -505,8 +508,11 @@ void IconbarTool::update(FbTk::Subject *subj) {
             return;
         } else if (subj == &(winsubj->win().workspaceSig())) {
             // we can ignore this signal if we're in ALLWINDOWS mode
-            if (mode() == ALLWINDOWS)
+            // unless the window was focused and has nothing to revert to
+            if (mode() == ALLWINDOWS || mode() == ICONS || mode() == NOICONS) {
+                m_focus_timer.start();
                 return;
+            }
 
             // workspace changed for this window, and if it's not on current workspace we remove it
             if (m_screen.currentWorkspaceID() != winsubj->win().workspaceNumber()) {
@@ -543,7 +549,7 @@ void IconbarTool::update(FbTk::Subject *subj) {
     bool remove_all = false; // if we should readd all windows
 
     if (subj != 0 && typeid(*subj) == typeid(BScreen::ScreenSubject) &&
-        mode() != ALLWINDOWS && mode() != ICONS ) {
+        mode() != ALLWINDOWS && mode() != ICONS && mode() != NOICONS) {
         BScreen::ScreenSubject *screen_subj = static_cast<BScreen::ScreenSubject *>(subj);
         // current workspace sig
         if (&m_screen.currentWorkspaceSig() == screen_subj ) {
@@ -838,11 +844,15 @@ bool IconbarTool::checkDuplicate(FluxboxWindow &win) {
 
 void IconbarTool::timedRender() {
     WinClient *client = FocusControl::focusedWindow();
-    if (client == 0 || client->fbwindow() == 0)
+    IconButton *current_button = static_cast<IconButton *>(m_icon_container.selected());
+
+    if (client == 0 || client->fbwindow() == 0) {
+        if (current_button != 0)
+            renderButton(*current_button);
         return;
+    }
 
     IconButton *button = findButton(*client->fbwindow());
-    IconButton *current_button = static_cast<IconButton *>(m_icon_container.selected());
     // if old window is the same as the new focused window then ignore this render
     // else render old client and new client
     if (button == current_button)
-- 
cgit v0.11.2