From fdeb749da7b63624457ec6cd006dfcb373e6ebfe Mon Sep 17 00:00:00 2001
From: dan <dan>
Date: Thu, 27 Feb 2003 07:42:46 +0000
Subject: [PATCH] Autoarrange icons after unhiding an application that has
 miniaturized windows, else they may cover other icons which are now in the
 positions they have occupied before hiding

---
 ChangeLog     |  3 +++
 TODO          |  3 +++
 WINGs/wtext.c | 11 ++++++-----
 src/actions.c |  4 +++-
 4 files changed, 15 insertions(+), 6 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index ed7fa9f0..370d3272 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -59,6 +59,9 @@ Changes since version 0.80.2:
 - Fixed a bug with not showing shaded window frames when the application is
   hidden and another window belonging to the app is selected in the window
   list menu
+- Autoarrange icons after unhiding an application that has miniaturized
+  windows, else they may cover other icons which are now in the positions
+  they have occupied before hiding
 
 
 Changes since version 0.80.1:
diff --git a/TODO b/TODO
index 8a8f4f8c..49aea02b 100644
--- a/TODO
+++ b/TODO
@@ -14,6 +14,9 @@ Do ASAP:
 
 Need to do:
 ===========
+- fix scaling/maxscaling code. it seem to always maxscale
+- impelment a generic pixmap type, which receives attributes
+  (scaled/maxscaled/centered/tiled, smoothed, size/position, ...)
 - allow user to select/restore default root menu from wprefs
 - support for X11R6.4 extension for getting extra visual info in wrlib's
   automatic best context guessing
diff --git a/WINGs/wtext.c b/WINGs/wtext.c
index 9cbc7e7a..6bd0219b 100644
--- a/WINGs/wtext.c
+++ b/WINGs/wtext.c
@@ -2432,15 +2432,16 @@ handleTextKeyPress(Text *tPtr, XEvent *event)
             Bool i = !tPtr->flags.rulerShown; 
             WMShowTextRuler(tPtr, i);
             tPtr->flags.rulerShown = i;
-         }   
-        else if (control_pressed && *buffer == '') 
+        } else if (control_pressed && *buffer == '\a') {
             XBell(tPtr->view->screen->display, 0);
-    else    
-        WMRelayToNextResponder(tPtr->view, event);
+        } else {
+            WMRelayToNextResponder(tPtr->view, event);
+        }
     }
 
-    if (!control_pressed && tPtr->flags.ownsSelection) 
+    if (!control_pressed && tPtr->flags.ownsSelection) {
         releaseSelection(tPtr);
+    }
 }
 
 
diff --git a/src/actions.c b/src/actions.c
index 50d6c953..019f3d86 100644
--- a/src/actions.c
+++ b/src/actions.c
@@ -1361,6 +1361,7 @@ wUnhideApplication(WApplication *wapp, Bool miniwindows, Bool bringToCurrentWS)
     WScreen *scr;
     WWindow *wlist, *next;
     WWindow *focused=NULL;
+    Bool shouldArrangeIcons = False;
 
     if (!wapp)
 	return;
@@ -1398,6 +1399,7 @@ wUnhideApplication(WApplication *wapp, Bool miniwindows, Bool bringToCurrentWS)
                     wlist->frame->workspace == scr->current_workspace) {
                     wDeiconifyWindow(wlist);
                 }
+                shouldArrangeIcons = True;
                 WMPostNotificationName(WMNChangedState, wlist, "hide");
             } else if (wlist->flags.shaded) {
                 if (bringToCurrentWS)
@@ -1437,7 +1439,7 @@ wUnhideApplication(WApplication *wapp, Bool miniwindows, Bool bringToCurrentWS)
         wSetFocusTo(scr, focused);
     }
     wapp->last_focused = NULL;
-    if (wPreferences.auto_arrange_icons) {
+    if (shouldArrangeIcons || wPreferences.auto_arrange_icons) {
         wArrangeIcons(scr, True);
     }
 #ifdef HIDDENDOT