several fixes for background option in styles

4 files changed, 147 insertions, 98 deletions

diff --git a/src/FbTk/Theme.hh b/src/FbTk/Theme.hh
index 70b782a..df2cea7 100644
--- a/src/FbTk/Theme.hh
+++ b/src/FbTk/Theme.hh
@@ -68,11 +68,11 @@ public:
     /// specialized
     void setDefaultValue(); 
     /// specialized
-    void setFromString(const char *strval);
+    virtual void setFromString(const char *strval);
     /// specialized
     // name and altname may be different to the primary ones (e.g. from fallback)
     // if they are null, then the original name is used
-    void load(const std::string *name = 0, const std::string *altname = 0);
+    virtual void load(const std::string *name = 0, const std::string *altname = 0);
     /**
        @name access operators
     */
diff --git a/src/RootTheme.cc b/src/RootTheme.cc
index 6847fc2..eeb95f9 100644
--- a/src/RootTheme.cc
+++ b/src/RootTheme.cc
@@ -48,7 +48,8 @@ using std::string;
 class BackgroundItem: public FbTk::ThemeItem<FbTk::Texture> {
 public:
     BackgroundItem(FbTk::Theme &tm, const std::string &name, const std::string &altname):
-        FbTk::ThemeItem<FbTk::Texture>(tm, name, altname) {
+        FbTk::ThemeItem<FbTk::Texture>(tm, name, altname),
+        m_changed(false), m_loaded(false) {
         
     }
 
@@ -56,6 +57,9 @@ public:
         const string &m_name = (o_name == 0) ? name() : *o_name;
         const string &m_altname = (o_altname == 0) ? altName() : *o_altname;
 
+        // if we got this far, then the background was loaded
+        m_loaded = true;
+
         // create subnames
         string color_name(FbTk::ThemeManager::instance().
                           resourceValue(m_name + ".color", m_altname + ".Color"));
@@ -63,9 +67,59 @@ public:
                             resourceValue(m_name + ".colorTo", m_altname + ".ColorTo"));
         string pixmap_name(FbTk::ThemeManager::instance().
                            resourceValue(m_name + ".pixmap", m_altname + ".Pixmap"));
+        string mod_x(FbTk::ThemeManager::instance().
+                     resourceValue(m_name + ".modX", m_altname + ".ModX"));
+        string mod_y(FbTk::ThemeManager::instance().
+                     resourceValue(m_name + ".modY", m_altname + ".ModY"));
+
+        // validate mod_x and mod_y
+        if (mod_x.length() > 2)
+            mod_x.erase(2,mod_x.length()); // shouldn't be longer than 2 digits
+        if (mod_y.length() > 2)
+            mod_y.erase(2,mod_y.length()); // ditto
+        // should be integers
+        if (!mod_x.length() || mod_x[0] < '0' || mod_x[0] > '9' ||
+            (mod_x.length() == 2 && (mod_x[1] < '0' || mod_x[1] > '9')))
+            mod_x = "1";
+        if (!mod_y.length() || mod_y[0] < '0' || mod_y[0] > '9' ||
+            (mod_y.length() == 2 && (mod_y[1] < '0' || mod_y[1] > '9')))
+            mod_y = "1";
+
+        // check if any of our values have changed
+        if (mod_x != m_mod_x) {
+            m_changed = true;
+            m_mod_x = mod_x;
+        }
+        if (mod_y != m_mod_y) {
+            m_changed = true;
+            m_mod_y = mod_y;
+        }
+        // these aren't quite right, but I don't care
+        if (color_name != m_color) {
+            m_changed = true;
+            m_color = color_name;
+        }
+        if (colorto_name != m_color_to) {
+            m_changed = true;
+            m_color_to = colorto_name;
+        }
+
+        // remove whitespace from filename
+        FbTk::StringUtil::removeFirstWhitespace(pixmap_name);
+        FbTk::StringUtil::removeTrailingWhitespace(pixmap_name);
+
+        if (mod_x != m_mod_x || mod_y != m_mod_y || pixmap_name != m_filename ||
+                color_name != m_color || colorto_name != m_color_to) {
+            m_changed = true;
+            m_mod_x = mod_x;
+            m_mod_y = mod_y;
+            m_filename = pixmap_name;
+
+            // these aren't quite right because of defaults set below
+            m_color = color_name;
+            m_color_to = colorto_name;
+        }
 
-        m_color = color_name;
-        m_color_to = colorto_name;
         // set default value if we failed to load colors
         if (!(*this)->color().setFromString(color_name.c_str(),
                                             theme().screenNum()))
@@ -79,11 +133,6 @@ public:
         if (((*this)->type() & FbTk::Texture::SOLID) != 0 && ((*this)->type() & FbTk::Texture::FLAT) == 0)
             (*this)->calcHiLoColors(theme().screenNum());
 
-        // remove whitespace and set filename
-        FbTk::StringUtil::removeFirstWhitespace(pixmap_name);
-        FbTk::StringUtil::removeTrailingWhitespace(pixmap_name);
-        m_filename = pixmap_name;
-
         // we dont load any pixmap, using external command to set background pixmap
         (*this)->pixmap() = 0;
     }
@@ -96,9 +145,17 @@ public:
     const std::string &options() const { return m_options; }
     const std::string &colorString() const { return m_color; }
     const std::string &colorToString() const { return m_color_to; }
+    const std::string &modX() const { return m_mod_x; }
+    const std::string &modY() const { return m_mod_y; }
+    bool changed() const { return m_changed; }
+    bool loaded() const { return m_loaded; }
+    void setApplied() { m_changed = false; }
+    void unsetLoaded() { m_loaded = false; }
 private:
     std::string m_filename, m_options;
     std::string m_color, m_color_to;
+    std::string m_mod_x, m_mod_y;
+    bool m_changed, m_loaded;
 };
 
 
@@ -108,9 +165,7 @@ RootTheme::RootTheme(const std::string &root_command,
     m_background(new BackgroundItem(*this, "background", "Background")),
     m_opgc(RootWindow(FbTk::App::instance()->display(), image_control.screenNumber())),
     m_root_command(root_command),
-    m_image_ctrl(image_control),
-    m_already_set(false),
-    m_background_loaded(true) {
+    m_image_ctrl(image_control) {
 
     Display *disp = FbTk::App::instance()->display();
     m_opgc.setForeground(WhitePixel(disp, screenNum())^BlackPixel(disp, screenNum()));
@@ -129,7 +184,7 @@ bool RootTheme::fallback(FbTk::ThemeItem_base &item) {
     // we can deal with it in reconfigureTheme()
     if (item.name() == "background") {
         // mark no background loaded
-        m_background_loaded = false;
+        m_background->unsetLoaded();
         return true;
     }
     return false;
@@ -138,10 +193,19 @@ bool RootTheme::fallback(FbTk::ThemeItem_base &item) {
 void RootTheme::reconfigTheme() {
     _FB_USES_NLS;
 
-    if (m_already_set)
+    if (!m_background->loaded()) {
+        cerr<<"Fluxbox: "<<
+            _FB_CONSOLETEXT(Common, BackgroundWarning,
+                    "There is no background option specified in this style."
+                    " Please consult the manual or read the FAQ.",
+                    "Background missing warning")<<endl;
+        return;
+    }
+
+    if (!m_background->changed())
         return;
-    else
-        m_already_set = true;
+
+    m_background->setApplied();
 
     // if user specified background in the config then use it
     // instead of style background
@@ -151,6 +215,11 @@ void RootTheme::reconfigTheme() {
         return;
     }
 
+    // style doesn't wish to change the background
+    if (strstr(m_background->options().c_str(), "none") != 0)
+        return;
+
+
     //
     // Else parse background from style 
     //
@@ -158,84 +227,70 @@ void RootTheme::reconfigTheme() {
     // root window helper
     FbRootWindow rootwin(screenNum());
 
-    // if the background theme item was not loaded
-        
-    if (!m_background_loaded) {
-        cerr<<"Fluxbox: "<<
-            _FB_CONSOLETEXT(Common, BackgroundWarning,
-                    "There is no background option specified in this style."
-                    " Please consult the manual or read the FAQ.",
-                    "Background missing warning")<<endl;
-    } else {
-        // handle background option in style
-        std::string filename = m_background->filename();
-        FbTk::StringUtil::removeTrailingWhitespace(filename);
-        FbTk::StringUtil::removeFirstWhitespace(filename);
-        // if background argument is a file then
-        // parse image options and call image setting 
-        // command specified in the resources
-        filename = FbTk::StringUtil::expandFilename(filename);
-        if (FbTk::FileUtil::isRegularFile(filename.c_str())) {
-            // parse options
-            std::string options;
-            if (strstr(m_background->options().c_str(), "tiled") != 0)
-                options += "-t ";
-            if (strstr(m_background->options().c_str(), "centered") != 0)
-                options += "-c ";
-            if (strstr(m_background->options().c_str(), "aspect") != 0)
-                options += "-a ";
+    // handle background option in style
+    std::string filename = m_background->filename();
+    FbTk::StringUtil::removeTrailingWhitespace(filename);
+    FbTk::StringUtil::removeFirstWhitespace(filename);
+    // if background argument is a file then
+    // parse image options and call image setting 
+    // command specified in the resources
+    filename = FbTk::StringUtil::expandFilename(filename);
+    if (FbTk::FileUtil::isRegularFile(filename.c_str())) {
+        // parse options
+        std::string options;
+        if (strstr(m_background->options().c_str(), "tiled") != 0)
+            options += "-t ";
+        if (strstr(m_background->options().c_str(), "centered") != 0)
+            options += "-c ";
+        if (strstr(m_background->options().c_str(), "aspect") != 0)
+            options += "-a ";
             
-            // compose wallpaper application "fbsetbg" with argumetns
-            std::string commandargs = "fbsetbg " + options + " " + filename;
-
-            // call command with options
-            FbCommands::ExecuteCmd exec(commandargs, screenNum());
-            exec.execute();
-
-        } else if (FbTk::FileUtil::isDirectory(filename.c_str()) &&
-                strstr(m_background->options().c_str(), "random") != 0) {
-            std::string commandargs = "fbsetbg -r " + filename;
-            FbCommands::ExecuteCmd exec(commandargs, screenNum());
-            exec.execute();
-        } else {
-            // render normal texture with fbsetroot
-
-
-            // Make sure the color strings are valid, 
-            // so we dont pass any `commands` that can be executed
-            bool color_valid = 
-                FbTk::Color::validColorString(m_background->colorString().c_str(), 
-                                              screenNum());
-            bool color_to_valid = 
-                FbTk::Color::validColorString(m_background->colorToString().c_str(), 
-                                              screenNum());
-
-            std::string options;
-            if (color_valid)
-                options += "-foreground '" + m_background->colorString() + "' ";
-            if (color_to_valid)
-                options += "-background '" + m_background->colorToString() + "' ";
-
-            if ((*m_background)->type() & FbTk::Texture::SOLID && color_valid)
-                options += "-solid '" + m_background->colorString() + "' ";
-
-            if ((*m_background)->type() & FbTk::Texture::GRADIENT) {
-
-                if (color_valid)
-                    options += "-from '" + m_background->colorString() + "' ";
-                if (color_to_valid)
-                    options += "-to '" + m_background->colorToString() + "' ";
-
-                options += "-gradient '" + m_background->options() + "'";
-            }
-
-            std::string commandargs = "fbsetroot " + options;
-
-            FbCommands::ExecuteCmd exec(commandargs, screenNum());
-            exec.execute();
+        // compose wallpaper application "fbsetbg" with argumetns
+        std::string commandargs = "fbsetbg " + options + " " + filename;
+
+        // call command with options
+        FbCommands::ExecuteCmd exec(commandargs, screenNum());
+        exec.execute();
+
+    } else if (FbTk::FileUtil::isDirectory(filename.c_str()) &&
+            strstr(m_background->options().c_str(), "random") != 0) {
+        std::string commandargs = "fbsetbg -r " + filename;
+        FbCommands::ExecuteCmd exec(commandargs, screenNum());
+        exec.execute();
+    } else {
+        // render normal texture with fbsetroot
+
+
+        // Make sure the color strings are valid, 
+        // so we dont pass any `commands` that can be executed
+        bool color_valid = 
+            FbTk::Color::validColorString(m_background->colorString().c_str(), 
+                                          screenNum());
+        bool color_to_valid = 
+            FbTk::Color::validColorString(m_background->colorToString().c_str(),
+                                          screenNum());
+
+        std::string options;
+        if (color_valid)
+            options += "-foreground '" + m_background->colorString() + "' ";
+        if (color_to_valid)
+            options += "-background '" + m_background->colorToString() + "' ";
+
+        if (strstr(m_background->options().c_str(), "mod") != 0)
+            options += "-mod " + m_background->modX() + " " + m_background->modY();
+        else if ((*m_background)->type() & FbTk::Texture::SOLID && color_valid)
+            options += "-solid '" + m_background->colorString() + "' ";
+
+        else if ((*m_background)->type() & FbTk::Texture::GRADIENT) {
+            options += "-gradient '" + m_background->options() + "'";
         }
 
-        rootwin.clear();
+        std::string commandargs = "fbsetroot " + options;
+
+        FbCommands::ExecuteCmd exec(commandargs, screenNum());
+        exec.execute();
     }
 
+    rootwin.clear();
+
 }
diff --git a/src/RootTheme.hh b/src/RootTheme.hh
index 24e9d36..aee6d85 100644
--- a/src/RootTheme.hh
+++ b/src/RootTheme.hh
@@ -51,8 +51,6 @@ public:
 
     bool fallback(FbTk::ThemeItem_base &item);
     void reconfigTheme();
-    // little hack to deal with reconfigures -- should be fixed
-    void setLoaded() { m_background_loaded = true; m_already_set = false; }
 
     GC opGC() const { return m_opgc.gc(); }
 
@@ -68,8 +66,6 @@ private:
     FbTk::GContext m_opgc;
     const std::string &m_root_command;
     FbTk::ImageControl &m_image_ctrl; ///< image control for rendering background texture
-    bool m_already_set;
-    bool m_background_loaded; ///< whether or not the background is present in the style file
 
 };
 
diff --git a/src/Screen.cc b/src/Screen.cc
index 89400fe..a82307d 100644
--- a/src/Screen.cc
+++ b/src/Screen.cc
@@ -913,8 +913,6 @@ void BScreen::reconfigure() {
     // notify objects that the screen is reconfigured
     m_reconfigure_sig.notify();
 
-    m_root_theme->setLoaded();
-
     // Reload style
     FbTk::ThemeManager::instance().load(fluxbox->getStyleFilename(),
                                         fluxbox->getStyleOverlayFilename(),