transaction: Move centering to view_center_surface

This will allow us to reuse it for centering elsewhere.

3 files changed, 12 insertions, 12 deletions

diff --git a/include/sway/tree/view.h b/include/sway/tree/view.h
index e071e6c9..18dc2019 100644
--- a/include/sway/tree/view.h
+++ b/include/sway/tree/view.h
@@ -317,6 +317,7 @@ void view_map(struct sway_view *view, struct wlr_surface *wlr_surface,
 void view_unmap(struct sway_view *view);
 
 void view_update_size(struct sway_view *view, int width, int height);
+void view_center_surface(struct sway_view *view);
 
 void view_child_init(struct sway_view_child *child,
         const struct sway_view_child_impl *impl, struct sway_view *view,
diff --git a/sway/desktop/transaction.c b/sway/desktop/transaction.c
index 2d71ad2b..ead662f9 100644
--- a/sway/desktop/transaction.c
+++ b/sway/desktop/transaction.c
@@ -255,18 +255,7 @@ static void apply_container_state(struct sway_container *container,
         // the container. This is important for fullscreen views which
         // refuse to resize to the size of the output.
         if (view && view->surface) {
-                if (view->geometry.width < container->current.content_width) {
-                        container->surface_x = container->current.content_x +
-                                (container->current.content_width - view->geometry.width) / 2;
-                } else {
-                        container->surface_x = container->current.content_x;
-                }
-                if (view->geometry.height < container->current.content_height) {
-                        container->surface_y = container->current.content_y +
-                                (container->current.content_height - view->geometry.height) / 2;
-                } else {
-                        container->surface_y = container->current.content_y;
-                }
+                view_center_surface(view);
         }
 
         if (!container->node.destroying) {
diff --git a/sway/tree/view.c b/sway/tree/view.c
index 3b1e67ea..f4ae8171 100644
--- a/sway/tree/view.c
+++ b/sway/tree/view.c
@@ -887,6 +887,16 @@ void view_update_size(struct sway_view *view, int width, int height) {
         }
 }
 
+void view_center_surface(struct sway_view *view) {
+        struct sway_container *con = view->container;
+        // We always center the current coordinates rather than the next, as the
+        // geometry immediately affects the currently active rendering.
+        con->surface_x = fmax(con->current.content_x, con->current.content_x +
+                        (con->current.content_width - view->geometry.width) / 2);
+        con->surface_y = fmax(con->current.content_y, con->current.content_y +
+                        (con->current.content_height - view->geometry.height) / 2);
+}
+
 static const struct sway_view_child_impl subsurface_impl;
 
 static void subsurface_get_root_coords(struct sway_view_child *child,