Bug#576082: apvlv: should not force font color of statusbar

[recoverym4n]

Package: apvlv
Version: 0.0.9.3-1
Severity: wishlist


Current unstable version (it was trivial to backport it to lenny)
forces font color of it's 'statusbar' to black most of the time.

Such behaviour is inconsistent with font color of command bar, for
instance. Much more annoying the fact that 'statusbar' text is 
invisible when one uses dark GUI theme.

Patch attached to this bugreport tries to fix this inconsistency
more GTK-way. Hope it'll find it's way upstream, or at least to
next Debian version of this package.

-- System Information:
Debian Release: 5.0.4
  APT prefers stable
  APT policy: (500, 'stable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.26-2-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

Versions of packages apvlv depends on:
ii  libatk1.0-0           1.22.0-1           The ATK accessibility toolkit
ii  libc6                 2.7-18lenny2       GNU C Library: Shared libraries
ii  libcairo2             1.6.4-7            The Cairo 2D vector graphics libra
ii  libgcc1               1:4.3.2-1.1        GCC support library
ii  libglib2.0-0          2.16.6-3           The GLib library of C routines
ii  libgtk2.0-0           2.12.12-1~lenny1   The GTK+ graphical user interface 
ii  libpango1.0-0         1.20.5-5+lenny1    Layout and rendering of internatio
ii  libpoppler-glib3      0.8.7-3.1          PDF rendering library (GLib-based 
ii  libstdc++6            4.3.2-1.1          The GNU Standard C++ Library v3

apvlv recommends no packages.

apvlv suggests no packages.

-- debconf-show failed

--- a/src/ApvlvDir.cpp	2010-03-31 18:20:22.669950694 +0400
+++ b/src/ApvlvDir.cpp	2010-03-31 18:24:34.988831428 +0400
@@ -567,24 +567,10 @@
 
   void ApvlvDirStatus::active (bool act)
   {
-    GdkColor c;
-
-    if (act)
-      {
-	c.red = 300;
-	c.green = 300;
-	c.blue = 300;
-      }
-    else
-      {
-	c.red = 30000;
-	c.green = 30000;
-	c.blue = 30000;
-      }

     for (unsigned int i = 0; i < AD_STATUS_SIZE; ++i)
       {
-	gtk_widget_modify_fg (mStlab[i], GTK_STATE_NORMAL, &c);
+    gtk_widget_modify_fg (mStlab[i], (act) ? GTK_STATE_NORMAL : GTK_STATE_INSENSITIVE , NULL);
       }
   }
 
--- a/src/ApvlvDoc.cpp	2010-03-31 18:20:20.305456961 +0400
+++ b/src/ApvlvDoc.cpp	2010-03-31 18:23:50.325456608 +0400
@@ -1988,24 +1988,10 @@
 
   void ApvlvDocStatus::active (bool act)
   {
-    GdkColor c;
-
-    if (act)
-      {
-	c.red = 300;
-	c.green = 300;
-	c.blue = 300;
-      }
-    else
-      {
-	c.red = 30000;
-	c.green = 30000;
-	c.blue = 30000;
-      }

     for (unsigned int i = 0; i < AD_STATUS_SIZE; ++i)
       {
-	gtk_widget_modify_fg (mStlab[i], GTK_STATE_NORMAL, &c);
+   gtk_widget_modify_fg (mStlab[i], (act) ? GTK_STATE_NORMAL : GTK_STATE_INSENSITIVE , NULL);
       }
   }