Merge branch 'master' into next

tdemin · Apr 2, 2013 · 0b000b1 · 0b000b1
2 parents 2926ddb + 4f72c5c
commit 0b000b1
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 7 deletions.
diff --git a/src/con.c b/src/con.c
@@ -1087,6 +1087,12 @@ Rect con_border_style_rect(Con *con) {
     } else {
         result = (Rect){border_width, border_width, -(2 * border_width), -(2 * border_width)};
     }
+
+    /* Floating windows are never adjacent to any other window, so
+       don’t hide their border(s). This prevents bug #998. */
+    if (con_is_floating(con))
+      return result;
+
     if (borders_to_hide & ADJ_LEFT_SCREEN_EDGE) {
         result.x -= border_width;
         result.width += border_width;

diff --git a/testcases/t/194-regress-floating-size.t b/testcases/t/194-regress-floating-size.t
@@ -19,7 +19,7 @@
 # ticket #770, bug still present in commit ae88accf6fe3817ff42d0d51be1965071194766e
 use i3test i3_autostart => 0;
 
-sub test_with_new_window_value {
+sub test_with_config {
     my ($value) = @_;
 
     my $config = <<EOT;
@@ -28,8 +28,8 @@ font -misc-fixed-medium-r-normal--13-120-75-75-C-70-iso10646-1
 EOT
 
     if (defined($value)) {
-        $config .= "new_window $value\n";
-        diag("testing with new_window $value");
+        $config .= "$value\n";
+        diag("testing with $value");
     } else {
         diag("testing without new_window");
     }
@@ -49,9 +49,10 @@ EOT
     exit_gracefully($pid);
 }
 
-test_with_new_window_value(undef);
-test_with_new_window_value('1pixel');
-test_with_new_window_value('normal');
-test_with_new_window_value('none');
+test_with_config(undef);
+test_with_config('new_window 1pixel');
+test_with_config('new_window normal');
+test_with_config('new_window none');
+test_with_config('hide_edge_borders both');
 
 done_testing;