add support for these...

menu.border.color
window.active.border.color
window.inactive.border.color

Bear2 has a menu color now, the old themerc had that specified

diff --git a/themes/Bear2/openbox-3/themerc.xml b/themes/Bear2/openbox-3/themerc.xml
index 8885c4cd8034ff6a6e9e943ff882199f3cab3e3c..68b0adac85e4d73d49c831634f51864cbbfaff31 100644 (file)
--- a/themes/Bear2/openbox-3/themerc.xml
+++ b/themes/Bear2/openbox-3/themerc.xml
@@ -122,7 +122,7 @@
     <justify>center</justify>
   </window>
   <menu>
-    <border r="78" g="78" b="78" a="255"/>
+    <border r="157" g="157" b="157" a="255"/>
     <justify>center</justify>
     <overlap>2</overlap>
     <title>

diff --git a/tools/themetoxml/themetoxml.c b/tools/themetoxml/themetoxml.c
index ae9972e20f15fce7d78e137aa3cfcf8269de4335..ace17bf8f824b2034bfd405d2cc193e5777dee1c 100644 (file)
--- a/tools/themetoxml/themetoxml.c
+++ b/tools/themetoxml/themetoxml.c
@@ -346,12 +346,20 @@ int main(int argc, char **argv)
         CONT3("dimensions", "window", "border", NUM(i));
         CONT3("dimensions", "menu", "border", NUM(i));
     }
+    if (read_int(db, "menu.border.width", &i))
+        CONT3("dimensions", "menu", "border", NUM(i));
 
     if (read_color(db, "border.color", &i, &j, &k)) {
         COLOR3("window", "active", "border", i, j, k, 255);
         COLOR3("window", "inactive", "border", i, j, k, 255);
         COLOR2("menu", "border", i, j, k, 255);
     }
+    if (read_color(db, "window.active.border.color", &i, &j, &k))
+        COLOR3("window", "active", "border", i, j, k, 255);
+    if (read_color(db, "window.inactive.border.color", &i, &j, &k))
+        COLOR3("window", "inactive", "border", i, j, k, 255);
+    if (read_color(db, "menu.border.color", &i, &j, &k))
+        COLOR2("menu", "border", i, j, k, 255);
 
     if (read_int(db, "window.client.padding.width", &i)) {
         ATTR3("dimensions", "window", "clientpadding", "x", NUM(i));