fix a few more bugs with recent reworking stuff

Some fixes could incidentally fix a few other bugs... the flaws were
pre-existing, I've just been weeding out a lot of duplication

4 files changed, 17 insertions, 14 deletions

diff --git a/src/FbTk/FbWindow.hh b/src/FbTk/FbWindow.hh
index f02c63e..b08df5e 100644
--- a/src/FbTk/FbWindow.hh
+++ b/src/FbTk/FbWindow.hh
@@ -116,7 +116,7 @@ public:
         XResizeWindow(s_display, m_window, width, height);
         m_width = width;
         m_height = height;
-        updateBackground(true);
+        updateBackground(false);
     }
 
     virtual inline void moveResize(int x, int y, unsigned int width, unsigned int height) {
@@ -127,7 +127,7 @@ public:
         m_y = y;
         m_width = width;
         m_height = height;
-        updateBackground(true);
+        updateBackground(false);
 
     }
     virtual void lower();
@@ -185,9 +185,11 @@ public:
     void setOpaque(unsigned char alpha);
 
     void setRenderer(FbWindowRenderer &renderer) { m_renderer = &renderer; }
-
     void sendConfigureNotify(int x, int y, unsigned int width, unsigned int height);
 
+    /// forces full background change, recalcing of alpha values if necessary
+    void updateBackground(bool only_if_alpha);
+
 protected:
     /// creates a window with x window client (m_window = client)
     explicit FbWindow(Window client);
@@ -204,8 +206,6 @@ private:
                 bool save_unders,
                 int depth, 
                 int class_type);
-    /// forces full background change, recalcing of alpha values if necessary
-    void updateBackground(bool only_if_alpha);
 
     const FbWindow *m_parent; ///< parent FbWindow
     int m_screen_num;  ///< screen num on which this window exist
diff --git a/src/FbTk/MenuSeparator.cc b/src/FbTk/MenuSeparator.cc
index a82ccdb..01aa24a 100644
--- a/src/FbTk/MenuSeparator.cc
+++ b/src/FbTk/MenuSeparator.cc
@@ -32,16 +32,19 @@ namespace FbTk {
 
 void MenuSeparator::draw(FbDrawable &drawable, 
                          const MenuTheme &theme,
-                         bool highlight,
+                         bool highlight, bool draw_foreground, bool draw_background,
                          int x, int y,
                          unsigned int width, unsigned int height) const {
-    const GContext &tgc =
-        (highlight ? theme.hiliteTextGC() :
-         (isEnabled() ? theme.frameTextGC() : theme.disableTextGC() ) );
 
-    drawable.drawRectangle(tgc.gc(),
-                           x + theme.bevelWidth() + height + 1, y + height / 2,
-        width - ((theme.bevelWidth() + height) * 2) - 1, 0);
+    if (draw_background) {
+        const GContext &tgc =
+            (highlight ? theme.hiliteTextGC() :
+             (isEnabled() ? theme.frameTextGC() : theme.disableTextGC() ) );
+
+        drawable.drawRectangle(tgc.gc(),
+                               x + theme.bevelWidth() + height + 1, y + height / 2,
+                               width - ((theme.bevelWidth() + height) * 2) - 1, 0);
+    }
 }
 
 }
diff --git a/src/FbTk/MenuSeparator.hh b/src/FbTk/MenuSeparator.hh
index 1cab4fc..3162193 100644
--- a/src/FbTk/MenuSeparator.hh
+++ b/src/FbTk/MenuSeparator.hh
@@ -33,7 +33,7 @@ class MenuSeparator: public MenuItem {
 public:
     virtual void draw(FbDrawable &drawable, 
                       const MenuTheme &theme,
-                      bool highlight,
+                      bool highlight, bool draw_foreground, bool draw_background,
                       int x, int y,
                       unsigned int width, unsigned int height) const;
 
diff --git a/src/FbTk/TextButton.cc b/src/FbTk/TextButton.cc
index 9a16c2c..e95b1c9 100644
--- a/src/FbTk/TextButton.cc
+++ b/src/FbTk/TextButton.cc
@@ -63,7 +63,7 @@ void TextButton::setJustify(FbTk::Justify just) {
 void TextButton::setText(const std::string &text) {
     if (m_text != text) {
         m_text = text;
-        parentMoved();
+        updateBackground(false);
         clear();
     }
 }