pauljd/vectorcraft
1
0
Fork 0
mirror of https://github.com/storytold/vectorcraft.git synced 2026-11-11 15:21:58 +00:00
Code Issues Packages Projects Releases Wiki Activity

address review: charge only blank lines to the line-break budget

Browse Source
This commit is contained in:
Dylan M. Taylor
2026-11-11 00:17:55 +00:00
parent cf52dda015
commit b3c0ad039e
1 changed files with 5 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
crates/svg/src/import/text.rs
+5 -4
View File
@@ -44,8 +44,8 @@ const SVG_NS: &str = "http://www.w3.org/2000/svg";
/// Most line breaks one absolutely positioned character adds before itself.
const MAX_LINE_GAP: f64 = 10_000.0;
/// Most line breaks the absolutely positioned characters of one text add in all: past it, a gap is
/// one break as wide as the gap.
/// Most blank lines (breaks past the first newline of each gap) the absolutely positioned
/// characters of one text add in all: past it, a gap is one break as wide as the gap.
const MAX_LINE_BREAKS: usize = 10_000;
/// A whitespace/comma separated list.
@@ -932,8 +932,9 @@ fn lines(
// Whole multiples of the auto leading add blank lines (not without limit: a far-off y
// mustn't ask for billions); other spacing is one break.
let count = (gap / lead).round().clamp(1.0, MAX_LINE_GAP);
let count = if (gap / count - lead).abs() <= lead * 0.05 && added + count as usize <= MAX_LINE_BREAKS { count } else { 1.0 };
added += count as usize;
// The first newline just ends the previous line: only the others are blank lines.
let count = if (gap / count - lead).abs() <= lead * 0.05 && added + (count as usize - 1) <= MAX_LINE_BREAKS { count } else { 1.0 };
added += count as usize - 1;
breaks.push(Break { at: i, count: count as usize, leading: gap / count });
lines.push((i, ax + dx[i]));
(base, cur) = (ny, ny);