add support for transient windows in client patterns, and merge a few more changes from pre-devel

6 files changed, 28 insertions, 15 deletions

diff --git a/src/ClientPattern.cc b/src/ClientPattern.cc
index 9324bee..538b708 100644
--- a/src/ClientPattern.cc
+++ b/src/ClientPattern.cc
@@ -60,7 +60,7 @@ ClientPattern::ClientPattern():
     m_nummatches(0) {}
 
 // parse the given pattern (to end of line)
-ClientPattern::ClientPattern(const char *str):
+ClientPattern::ClientPattern(const char *str, bool default_no_transient):
     m_matchlimit(0),
     m_nummatches(0)
 {
@@ -107,6 +107,9 @@ ClientPattern::ClientPattern(const char *str):
                 prop = TITLE;
             } else if (strcasecmp(memstr.c_str(), "role") == 0) {
                 prop = ROLE;
+            } else if (strcasecmp(memstr.c_str(), "transient") == 0) {
+                prop = TRANSIENT;
+                default_no_transient = false;
             } else if (strcasecmp(memstr.c_str(), "maximized") == 0) {
                 prop = MAXIMIZED;
             } else if (strcasecmp(memstr.c_str(), "minimized") == 0) {
@@ -138,6 +141,9 @@ ClientPattern::ClientPattern(const char *str):
         had_error = true;
     }
 
+    if (default_no_transient)
+        had_error = !addTerm("no", TRANSIENT);
+
     if (!had_error) {
         // otherwise, we check for a number
         string number;
@@ -201,6 +207,9 @@ string ClientPattern::toString() const {
         case ROLE:
             pat.append("role=");
             break;
+        case TRANSIENT:
+            pat.append("transient=");
+            break;
         case MAXIMIZED:
             pat.append("maximized=");
             break;
@@ -307,6 +316,9 @@ string ClientPattern::getProperty(WinProperty prop,
     case ROLE:
         return client.getWMRole();
         break;
+    case TRANSIENT:
+        return client.isTransient() ? "yes" : "no";
+        break;
     case MAXIMIZED:
         return (fbwin && fbwin->isMaximized()) ? "yes" : "no";
         break;
diff --git a/src/ClientPattern.hh b/src/ClientPattern.hh
index 4a3cd0b..e5f4944 100644
--- a/src/ClientPattern.hh
+++ b/src/ClientPattern.hh
@@ -46,7 +46,7 @@ public:
      * apps file. the bool value returns the character at which
      * there was a parse problem, or -1.
      */
-    explicit ClientPattern(const char * str);
+    explicit ClientPattern(const char * str, bool default_no_transient = false);
 
     ~ClientPattern();
 
@@ -54,7 +54,7 @@ public:
     std::string toString() const;
 
     enum WinProperty {
-        TITLE, CLASS, NAME, ROLE,
+        TITLE, CLASS, NAME, ROLE, TRANSIENT,
         MAXIMIZED, MINIMIZED, SHADED, STUCK, FOCUSHIDDEN, ICONHIDDEN,
         WORKSPACE, HEAD, LAYER
     };
diff --git a/src/Focusable.hh b/src/Focusable.hh
index e02e0d7..cd2ad49 100644
--- a/src/Focusable.hh
+++ b/src/Focusable.hh
@@ -84,9 +84,12 @@ public:
     virtual const std::string &getWMClassClass() const { return m_class_name; }
     /// @return WM_CLASS name string (for pattern matching)
     virtual const std::string &getWMClassName() const { return m_instance_name; }
-    /// @return wm role string ( for pattern matching)
+    /// @return wm role string (for pattern matching)
     virtual std::string getWMRole() const { return "Focusable"; }
 
+    /// @return whether this window is a transient (for pattern matching)
+    virtual bool isTransient() const { return false; }
+
     // so we can make nice buttons, menu entries, etc.
     /// @return icon pixmap of the focusable
     virtual const FbTk::PixmapWithMask &icon() const { return m_icon; }
diff --git a/src/Remember.cc b/src/Remember.cc
index fbb4a2e..d8f1d83 100644
--- a/src/Remember.cc
+++ b/src/Remember.cc
@@ -339,6 +339,7 @@ Application * Remember::add(WinClient &winclient) {
     p->addTerm(win_class, ClientPattern::CLASS);
     if (!win_role.empty())
         p->addTerm(win_role, ClientPattern::ROLE);
+    p->addTerm(winclient.isTransient() ? "yes" : "no", ClientPattern::TRANSIENT);
     m_clients[&winclient] = app;
     p->addMatch();
     m_pats->push_back(make_pair(p, app));
@@ -594,7 +595,7 @@ void Remember::reconfigure() {
                                                              '[', ']');
 
                 if (pos > 0 && strcasecmp(key.c_str(), "app") == 0) {
-                    ClientPattern *pat = new ClientPattern(line.c_str() + pos);
+                    ClientPattern *pat = new ClientPattern(line.c_str() + pos, true);
                     if (!in_group) {
                         if ((err = pat->error()) == 0) {
                             Application *app = findMatchingPatterns(pat, old_pats, false);
@@ -620,7 +621,7 @@ void Remember::reconfigure() {
                 } else if (pos > 0 && strcasecmp(key.c_str(), "group") == 0) {
                     in_group = true;
                     if (line.find('(') != string::npos)
-                        pat = new ClientPattern(line.c_str() + pos);
+                        pat = new ClientPattern(line.c_str() + pos, true);
                 } else if (in_group) {
                     // otherwise assume that it is the start of the attributes
                     Application *app = 0;
@@ -1020,13 +1021,6 @@ void Remember::forgetAttrib(WinClient &winclient, Attribute attrib) {
 
 void Remember::setupFrame(FluxboxWindow &win) {
     WinClient &winclient = win.winClient();
-    // we don't touch the window if it is a transient
-    // of something else
-
-
-    if (winclient.transientFor())
-        return;
-
     Application *app = find(winclient);
     if (app == 0)
         return; // nothing to do
@@ -1123,8 +1117,7 @@ void Remember::setupFrame(FluxboxWindow &win) {
 void Remember::setupClient(WinClient &winclient) {
 
     // leave windows alone on restart
-    // don't apply settings to transient windows
-    if (winclient.screen().isRestart() || winclient.transientFor())
+    if (winclient.screen().isRestart())
         return;
 
     Application *app = find(winclient);
diff --git a/src/Window.cc b/src/Window.cc
index a5cf6fd..27bc7ec 100644
--- a/src/Window.cc
+++ b/src/Window.cc
@@ -3636,6 +3636,10 @@ std::string FluxboxWindow::getWMRole() const {
     return (m_client ? m_client->getWMRole() : "FluxboxWindow");
 }
 
+bool FluxboxWindow::isTransient() const {
+    return (m_client && m_client->isTransient());
+}
+
 int FluxboxWindow::normalX() const {
     if (maximized & MAX_HORZ)
         return m_old_pos_x;
diff --git a/src/Window.hh b/src/Window.hh
index d001284..3a87463 100644
--- a/src/Window.hh
+++ b/src/Window.hh
@@ -435,6 +435,7 @@ public:
     const std::string &getWMClassName() const; 
     const std::string &getWMClassClass() const;
     std::string getWMRole() const;
+    bool isTransient() const;
 
     inline int x() const { return frame().x(); }
     inline int y() const { return frame().y(); }