From 936d480267578d7e80675e78ec1880aaaaab72d6 Mon Sep 17 00:00:00 2001
From: Héctor Ramón Jiménez <hector@hecrj.dev>
Date: Fri, 1 Dec 2023 16:04:27 +0100
Subject: Clip text to `viewport` bounds instead of layout bounds

---
 tiny_skia/src/backend.rs  | 17 +++++++----------
 tiny_skia/src/geometry.rs | 15 +++++++++------
 2 files changed, 16 insertions(+), 16 deletions(-)

(limited to 'tiny_skia/src')

diff --git a/tiny_skia/src/backend.rs b/tiny_skia/src/backend.rs
index f2905b00..cc0f72d1 100644
--- a/tiny_skia/src/backend.rs
+++ b/tiny_skia/src/backend.rs
@@ -1,6 +1,6 @@
 use crate::core::{Background, Color, Gradient, Rectangle, Vector};
 use crate::graphics::backend;
-use crate::graphics::{Damage, Viewport};
+use crate::graphics::Viewport;
 use crate::primitive::{self, Primitive};
 
 use std::borrow::Cow;
@@ -361,11 +361,9 @@ impl Backend {
                 paragraph,
                 position,
                 color,
+                viewport,
             } => {
-                let physical_bounds =
-                    (Rectangle::new(*position, paragraph.min_bounds)
-                        + translation)
-                        * scale_factor;
+                let physical_bounds = (*viewport + translation) * scale_factor;
 
                 if !clip_bounds.intersects(&physical_bounds) {
                     return;
@@ -387,10 +385,9 @@ impl Backend {
                 editor,
                 position,
                 color,
+                viewport,
             } => {
-                let physical_bounds =
-                    (Rectangle::new(*position, editor.bounds) + translation)
-                        * scale_factor;
+                let physical_bounds = (*viewport + translation) * scale_factor;
 
                 if !clip_bounds.intersects(&physical_bounds) {
                     return;
@@ -418,9 +415,9 @@ impl Backend {
                 horizontal_alignment,
                 vertical_alignment,
                 shaping,
+                viewport,
             } => {
-                let physical_bounds =
-                    (primitive.bounds() + translation) * scale_factor;
+                let physical_bounds = (*viewport + translation) * scale_factor;
 
                 if !clip_bounds.intersects(&physical_bounds) {
                     return;
diff --git a/tiny_skia/src/geometry.rs b/tiny_skia/src/geometry.rs
index 1d14aa03..b73f84a9 100644
--- a/tiny_skia/src/geometry.rs
+++ b/tiny_skia/src/geometry.rs
@@ -109,15 +109,17 @@ impl Frame {
             Point::new(transformed[0].x, transformed[0].y)
         };
 
+        let bounds = Rectangle {
+            x: position.x,
+            y: position.y,
+            width: f32::INFINITY,
+            height: f32::INFINITY,
+        };
+
         // TODO: Use vectorial text instead of primitive
         self.primitives.push(Primitive::Text {
             content: text.content,
-            bounds: Rectangle {
-                x: position.x,
-                y: position.y,
-                width: f32::INFINITY,
-                height: f32::INFINITY,
-            },
+            bounds,
             color: text.color,
             size: text.size,
             line_height: text.line_height,
@@ -125,6 +127,7 @@ impl Frame {
             horizontal_alignment: text.horizontal_alignment,
             vertical_alignment: text.vertical_alignment,
             shaping: text.shaping,
+            viewport: bounds,
         });
     }
 
-- 
cgit 


From b526ce4958b28208395276dd4078ffe0d780e1d7 Mon Sep 17 00:00:00 2001
From: Héctor Ramón Jiménez <hector@hecrj.dev>
Date: Sat, 2 Dec 2023 15:53:02 +0100
Subject: Rename `viewport` to `clip_bounds`

---
 tiny_skia/src/backend.rs  | 15 +++++++++------
 tiny_skia/src/geometry.rs |  2 +-
 2 files changed, 10 insertions(+), 7 deletions(-)

(limited to 'tiny_skia/src')

diff --git a/tiny_skia/src/backend.rs b/tiny_skia/src/backend.rs
index cc0f72d1..3e9bd2a5 100644
--- a/tiny_skia/src/backend.rs
+++ b/tiny_skia/src/backend.rs
@@ -361,9 +361,10 @@ impl Backend {
                 paragraph,
                 position,
                 color,
-                viewport,
+                clip_bounds: text_clip_bounds,
             } => {
-                let physical_bounds = (*viewport + translation) * scale_factor;
+                let physical_bounds =
+                    (*text_clip_bounds + translation) * scale_factor;
 
                 if !clip_bounds.intersects(&physical_bounds) {
                     return;
@@ -385,9 +386,10 @@ impl Backend {
                 editor,
                 position,
                 color,
-                viewport,
+                clip_bounds: text_clip_bounds,
             } => {
-                let physical_bounds = (*viewport + translation) * scale_factor;
+                let physical_bounds =
+                    (*text_clip_bounds + translation) * scale_factor;
 
                 if !clip_bounds.intersects(&physical_bounds) {
                     return;
@@ -415,9 +417,10 @@ impl Backend {
                 horizontal_alignment,
                 vertical_alignment,
                 shaping,
-                viewport,
+                clip_bounds: text_clip_bounds,
             } => {
-                let physical_bounds = (*viewport + translation) * scale_factor;
+                let physical_bounds =
+                    (*text_clip_bounds + translation) * scale_factor;
 
                 if !clip_bounds.intersects(&physical_bounds) {
                     return;
diff --git a/tiny_skia/src/geometry.rs b/tiny_skia/src/geometry.rs
index b73f84a9..5f28b737 100644
--- a/tiny_skia/src/geometry.rs
+++ b/tiny_skia/src/geometry.rs
@@ -127,7 +127,7 @@ impl Frame {
             horizontal_alignment: text.horizontal_alignment,
             vertical_alignment: text.vertical_alignment,
             shaping: text.shaping,
-            viewport: bounds,
+            clip_bounds: Rectangle::with_size(Size::INFINITY),
         });
     }
 
-- 
cgit