make "force pseudotransparency" menu item work immediately

3 files changed, 25 insertions, 32 deletions

diff --git a/src/FbTk/Menu.cc b/src/FbTk/Menu.cc
index 9206ade..e7fc6d8 100644
--- a/src/FbTk/Menu.cc
+++ b/src/FbTk/Menu.cc
@@ -1099,14 +1099,18 @@ void Menu::keyPressEvent(XKeyEvent &event) {
 
 void Menu::reconfigure() {
 
-    if (FbTk::Transparent::haveComposite())
+    if (FbTk::Transparent::haveComposite()) {
         menu.window.setOpaque(alpha());
+        menu.title.setAlpha(255);
+        menu.frame.setAlpha(255);
+    } else {
+        menu.window.setOpaque(255);
+        menu.title.setAlpha(alpha());
+        menu.frame.setAlpha(alpha());
+    }
 
     m_need_update = true; // redraw items
 
-    menu.title.setAlpha(alpha());
-    menu.frame.setAlpha(alpha());
-
     menu.window.setBorderColor(theme().borderColor());
     menu.title.setBorderColor(theme().borderColor());
     menu.frame.setBorderColor(theme().borderColor());
diff --git a/src/FbTk/Transparent.cc b/src/FbTk/Transparent.cc
index 6563d5a..d80b2b2 100644
--- a/src/FbTk/Transparent.cc
+++ b/src/FbTk/Transparent.cc
@@ -112,6 +112,7 @@ namespace FbTk {
 bool Transparent::s_init = false;
 bool Transparent::s_render = false;
 bool Transparent::s_composite = false;
+bool Transparent::s_use_composite = false;
 
 void Transparent::init() {
     Display *disp = FbTk::App::instance()->display();
@@ -119,48 +120,35 @@ void Transparent::init() {
     int major_opcode, first_event, first_error;
     if (XQueryExtension(disp, "RENDER",
                         &major_opcode,
-                        &first_event, &first_error) == False) {
-        s_render = false;
-        s_composite = false;
-    } else { // we have RENDER support
+                        &first_event, &first_error)) {
+        // we have XRENDER support
         s_render = true;
 
         if (XQueryExtension(disp, "Composite",
                             &major_opcode,
-                            &first_event, &first_error) == False) {
-            s_composite = false;
-        } else { // we have Composite support
+                            &first_event, &first_error)) {
+            // we have Composite support
             s_composite = true;
+            s_use_composite = true;
         }
     }
     s_init = true;
 }
 
-void Transparent::usePseudoTransparent(bool no_composite) {
-    if (s_composite != no_composite)
-        return;
-
-    s_init = false;
-    init(); // only use render if we have it
-
-    if (no_composite)
-        s_composite = false;
+void Transparent::usePseudoTransparent(bool force) {
+    if (!s_init)
+        init();
+    s_use_composite = (!force && s_composite);
 }
 
 bool Transparent::haveComposite(bool for_real) {
-    if (for_real) {
-        Display *disp = FbTk::App::instance()->display();
-        int major_opcode, first_event, first_error;
-
-        return (XQueryExtension(disp, "Composite",
-                               &major_opcode,
-                               &first_event, &first_error) == True);
-    } else {
-        if (!s_init)
-            init();
+    if (!s_init)
+        init();
 
+    if (for_real)
         return s_composite;
-    }
+    else
+        return s_use_composite;
 }
 
 Transparent::Transparent(Drawable src, Drawable dest, unsigned char alpha, int screen_num):
diff --git a/src/FbTk/Transparent.hh b/src/FbTk/Transparent.hh
index 77e09c5..3720b33 100644
--- a/src/FbTk/Transparent.hh
+++ b/src/FbTk/Transparent.hh
@@ -53,7 +53,7 @@ public:
 
     static bool haveComposite(bool for_real = false);
     static bool haveRender() { if (!s_init) init(); return s_render; }
-    static void usePseudoTransparent(bool no_composite);
+    static void usePseudoTransparent(bool force);
 
 private:
     void freeAlpha();
@@ -67,6 +67,7 @@ private:
     static bool s_init;
     static bool s_render; ///< wheter we have RENDER support
     static bool s_composite; ///< wheter we have Composite support
+    static bool s_use_composite; ///< whether or not to use Composite
     static void init();
 };