Merge pull request #690 from zandrmartin/smartgap-hide-borders

redo smart_gaps/hide_edge_borders fix

1 files changed, 9 insertions, 5 deletions

diff --git a/sway/layout.c b/sway/layout.c
index 8bf4ce00..952bcf1c 100644
--- a/sway/layout.c
+++ b/sway/layout.c
@@ -567,26 +567,30 @@ void update_geometry(swayc_t *container) {
                 int border_right = container->border_thickness;
 
                 // handle hide_edge_borders
-                if (config->hide_edge_borders != E_NONE && gap <= 0) {
+                if (config->hide_edge_borders != E_NONE && (gap <= 0 || (config->smart_gaps && ws->children->length == 1))) {
                         swayc_t *output = swayc_parent_by_type(container, C_OUTPUT);
                         const struct wlc_size *size = wlc_output_get_resolution(output->handle);
 
                         if (config->hide_edge_borders == E_HORIZONTAL || config->hide_edge_borders == E_BOTH) {
-                                if (geometry.origin.x == 0) {
+                                if (geometry.origin.x == 0 || geometry.origin.x == container->x) {
+                                        // should work for swaybar at left
                                         border_left = 0;
                                 }
 
-                                if (geometry.origin.x + geometry.size.w == size->w) {
+                                if (geometry.origin.x + geometry.size.w == size->w || geometry.size.w == container->width) {
+                                        // should work for swaybar at right
                                         border_right = 0;
                                 }
                         }
 
                         if (config->hide_edge_borders == E_VERTICAL || config->hide_edge_borders == E_BOTH) {
-                                if (geometry.origin.y == 0) {
+                                if (geometry.origin.y == 0 || geometry.origin.y == container->y) {
+                                        // this works for swaybar at top
                                         border_top = 0;
                                 }
 
-                                if (geometry.origin.y + geometry.size.h == size->h) {
+                                if (geometry.origin.y + geometry.size.h == size->h || geometry.size.h == container->height) {
+                                        // this works for swaybar at bottom
                                         border_bottom = 0;
                                 }
                         }