unmapped-view-node: report bbox with logical size, not physical (#2671)

Author: ammen99

plugins/animate/unmapped-view-node.hpp

@@ -10,12 +10,14 @@ namespace wf
 class unmapped_view_snapshot_node : public wf::scene::node_t
 {
     wf::auxilliary_buffer_t snapshot;
+    wf::dimensions_t snapshot_logical_size;
     std::weak_ptr<wf::view_interface_t> _view;
 
   public:
     unmapped_view_snapshot_node(wayfire_view view) : node_t(false)
     {
         view->take_snapshot(snapshot);
+        snapshot_logical_size = wf::dimensions(view->get_surface_root_node()->get_bounding_box());
         _view = view->weak_from_this();
     }
 
@@ -24,7 +26,7 @@ class unmapped_view_snapshot_node : public wf::scene::node_t
         if (auto view = _view.lock())
         {
             auto current_bbox = view->get_surface_root_node()->get_bounding_box();
-            return wf::construct_box(wf::origin(current_bbox), snapshot.get_size());
+            return wf::construct_box(wf::origin(current_bbox), snapshot_logical_size);
         }
 
         return {0, 0, 0, 0};

src/view/view.cpp

@@ -96,11 +96,10 @@ void wf::view_interface_t::take_snapshot(wf::auxilliary_buffer_t& buffer)
     auto root_node = get_surface_root_node();
     const wf::geometry_t bbox = root_node->get_bounding_box();
     float scale = get_output()->handle->scale;
-
-    buffer.allocate({int(bbox.width * scale), int(bbox.height * scale)});
+    buffer.allocate(wf::dimensions(bbox), scale);
 
     wf::render_target_t target{buffer};
-    target.geometry = root_node->get_bounding_box();
+    target.geometry = bbox;
     target.scale    = scale;
 
     std::vector<scene::render_instance_uptr> instances;