chris/cryoglyph
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Move text area bounds outside run/glyph loops (#110)

Browse source
This commit is contained in:
Josh Groves 2024-07-29 11:59:26 -02:30 • committed by Héctor Ramón Jiménez
parent 6b566c5eb1
commit 40861cd933
No known key found for this signature in database
GPG key ID: 7CC46565708259A7
1 changed files with 19 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

25
src/text_render.rs
View file

@ -67,7 +67,25 @@ impl TextRenderer {
let resolution = viewport.resolution();
for text_area in text_areas {
for run in text_area.buffer.layout_runs() {
let bounds_min_x = text_area.bounds.left.max(0);
let bounds_min_y = text_area.bounds.top.max(0);
let bounds_max_x = text_area.bounds.right.min(resolution.width as i32);
let bounds_max_y = text_area.bounds.bottom.min(resolution.height as i32);
let is_run_visible = |run: &cosmic_text::LayoutRun| {
let start_y = (text_area.top + run.line_top) as i32;
let end_y = (text_area.top + run.line_top + run.line_height) as i32;
start_y <= bounds_max_y && bounds_min_y <= end_y
};
let layout_runs = text_area
.buffer
.layout_runs()
.skip_while(|run| !is_run_visible(run))
.take_while(is_run_visible);
for run in layout_runs {
for glyph in run.glyphs.iter() {
let physical_glyph =
glyph.physical((text_area.left, text_area.top), text_area.scale);
@ -189,11 +207,6 @@ impl TextRenderer {
let mut width = details.width as i32;
let mut height = details.height as i32;
let bounds_min_x = text_area.bounds.left.max(0);
let bounds_min_y = text_area.bounds.top.max(0);
let bounds_max_x = text_area.bounds.right.min(resolution.width as i32);
let bounds_max_y = text_area.bounds.bottom.min(resolution.height as i32);
// Starts beyond right edge or ends beyond left edge
let max_x = x + width;
if x > bounds_max_x || max_x < bounds_min_x {