With ...

   *FvwmButtons: (ActiveColorset 1)

... the button isn't redrawn on EnterNotify.

Se attached patch for a fix.


/Simon Griph
Index: NEWS
===================================================================
RCS file: /home/cvs/fvwm/fvwm/NEWS,v
retrieving revision 1.705
diff -u -r1.705 NEWS
--- NEWS        15 Jul 2007 12:39:39 -0000      1.705
+++ NEWS        22 Jul 2007 16:47:18 -0000
@@ -67,6 +67,8 @@
    - The conditon following a comma separator without whitespace
      padding was previously ignored if the presiding condition was
      multi-worded.
+   - FvwmButtons now handles container specific ActiveColorset
+     correctly when neither ActiveIcon nor ActiveTitle is used.
 
 -------------------------------------------------------------------
 
Index: modules/ChangeLog
===================================================================
RCS file: /home/cvs/fvwm/fvwm/modules/ChangeLog,v
retrieving revision 1.1260
diff -u -r1.1260 ChangeLog
--- modules/ChangeLog   15 Jul 2007 12:39:39 -0000      1.1260
+++ modules/ChangeLog   22 Jul 2007 16:47:25 -0000
@@ -1,3 +1,9 @@
+2007-07-22  Simon Griph  <simon(at)griph(dot)se>
+
+       * FvwmButtons/FvwmButtons.c (handle_new_position):
+       make container specific ActiveColorset work without
+       ActiveIcon/ActiveTitle.
+
 2007-06-17  Simon Griph  <simon(at)griph(dot)se>
 
        * FvwmAnimate/Makefile.am (LDADD):
Index: modules/FvwmButtons/FvwmButtons.c
===================================================================
RCS file: /home/cvs/fvwm/fvwm/modules/FvwmButtons/FvwmButtons.c,v
retrieving revision 1.205
diff -u -r1.205 FvwmButtons.c
--- modules/FvwmButtons/FvwmButtons.c   2 Jun 2007 22:27:55 -0000       1.205
+++ modules/FvwmButtons/FvwmButtons.c   22 Jul 2007 16:47:27 -0000
@@ -933,8 +933,11 @@
                        ActiveButton = b;
                        RedrawButton(tmp, DRAW_FORCE, NULL);
                }
-               if (b->flags.b_ActiveIcon || b->flags.b_ActiveTitle ||
-                   UberButton->c->flags.b_ActiveColorset)
+               if (
+                       b->flags.b_ActiveIcon ||
+                       b->flags.b_ActiveTitle ||
+                       b->flags.b_ActiveColorset ||
+                       UberButton->c->flags.b_ActiveColorset)
                {
                        ActiveButton = b;
                        redraw = True;

Reply via email to