M engines/openbox/openbox.c → engines/openbox/openbox.c

@@ -594,28 +594,30 @@ 
 static void layout_title(ObFrame *self)
 {
     const char *lc;
-    int x;
+    int x, sep;
     gboolean n, d, i, l, m ,c;
+
     n = d = i = l = m = c = FALSE;
+    sep = s_bevel + 1;
 
     /* figure out whats being shown, and the width of the label */
-    self->label_width = self->width - s_bevel * 2;
+    self->label_width = self->width - sep * 2;
     for (lc = themerc_titlebar_layout; *lc != '\0'; ++lc) {
 	switch (*lc) {
 	case 'N':
 	    if (!(self->frame.client->decorations & Decor_Icon)) break;
 	    n = TRUE;
-	    self->label_width -= BUTTON_SIZE + s_bevel;
+	    self->label_width -= BUTTON_SIZE + sep;
 	    break;
 	case 'D':
 	    if (!(self->frame.client->decorations & Decor_AllDesktops)) break;
 	    d = TRUE;
-	    self->label_width -= BUTTON_SIZE + s_bevel;
+	    self->label_width -= BUTTON_SIZE + sep;
 	    break;
 	case 'I':
 	    if (!(self->frame.client->decorations & Decor_Iconify)) break;
 	    i = TRUE;
-	    self->label_width -= BUTTON_SIZE + s_bevel;
+	    self->label_width -= BUTTON_SIZE + sep;
 	    break;
 	case 'L':
 	    l = TRUE;
@@ -623,12 +625,12 @@ break;
 	case 'M':
 	    if (!(self->frame.client->decorations & Decor_Maximize)) break;
 	    m = TRUE;
-	    self->label_width -= BUTTON_SIZE + s_bevel;
+	    self->label_width -= BUTTON_SIZE + sep;
 	    break;
 	case 'C':
 	    if (!(self->frame.client->decorations & Decor_Close)) break;
 	    c = TRUE;
-	    self->label_width -= BUTTON_SIZE + s_bevel;
+	    self->label_width -= BUTTON_SIZE + sep;
 	    break;
 	}
     }
@@ -667,50 +669,50 @@ XUnmapWindow(ob_display, self->close);
 	self->close_x = -1;
     }
 
-    x = s_bevel;
+    x = sep;
     for (lc = themerc_titlebar_layout; *lc != '\0'; ++lc) {
 	switch (*lc) {
 	case 'N':
 	    if (!n) break;
 	    self->icon_x = x;
 	    XMapWindow(ob_display, self->icon);
-	    XMoveWindow(ob_display, self->icon, x, s_bevel + 1);
-	    x += BUTTON_SIZE + s_bevel;
+	    XMoveWindow(ob_display, self->icon, x, sep + 1);
+	    x += BUTTON_SIZE + sep;
 	    break;
 	case 'D':
 	    if (!d) break;
 	    self->desk_x = x;
 	    XMapWindow(ob_display, self->desk);
 	    XMoveWindow(ob_display, self->desk, x, s_bevel + 1);
-	    x += BUTTON_SIZE + s_bevel;
+	    x += BUTTON_SIZE + sep;
 	    break;
 	case 'I':
 	    if (!i) break;
 	    self->iconify_x = x;
 	    XMapWindow(ob_display, self->iconify);
 	    XMoveWindow(ob_display, self->iconify, x, s_bevel + 1);
-	    x += BUTTON_SIZE + s_bevel;
+	    x += BUTTON_SIZE + sep;
 	    break;
 	case 'L':
 	    if (!l) break;
 	    self->label_x = x;
 	    XMapWindow(ob_display, self->label);
 	    XMoveWindow(ob_display, self->label, x, s_bevel);
-	    x += self->label_width + s_bevel;
+	    x += self->label_width + sep;
 	    break;
 	case 'M':
 	    if (!m) break;
 	    self->max_x = x;
 	    XMapWindow(ob_display, self->max);
 	    XMoveWindow(ob_display, self->max, x, s_bevel + 1);
-	    x += BUTTON_SIZE + s_bevel;
+	    x += BUTTON_SIZE + sep;
 	    break;
 	case 'C':
 	    if (!c) break;
 	    self->close_x = x;
 	    XMapWindow(ob_display, self->close);
 	    XMoveWindow(ob_display, self->close, x, s_bevel + 1);
-	    x += BUTTON_SIZE + s_bevel;
+	    x += BUTTON_SIZE + sep;
 	    break;
 	}
     }