johanels/penpot
1
0
Fork 0
mirror of https://github.com/penpot/penpot.git synced 2025-12-12 06:24:17 +01:00
Code Issues Actions 9 Packages Projects Releases Wiki Activity

🐛 Fix underline not matching spacing/thickness

Browse Source
This commit is contained in:
Belén Albeza
2025-10-20 17:44:09 +02:00
parent b0e4257e56
commit 87221eb7db
1 changed files with 13 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
render-wasm/src/render/text.rs
View File

@@ -289,12 +289,21 @@ fn calculate_decoration_metrics(
.abs() .abs()
.max(font_metrics.x_height.abs()); .max(font_metrics.x_height.abs());
let min_thickness = (font_size * 0.06).max(1.0); let min_thickness = (font_size * 0.06).max(1.0);
let thickness = font_metrics
.underline_thickness() // Magic numbers for line thickness partially based on Chromium
.unwrap_or(1.0) // (see https://source.chromium.org/chromium/chromium/src/+/main:ui/gfx/render_text.cc
let raw_font_size = style_metric.text_style.font_size();
let thickness_factor = raw_font_size.powf(0.4) * 6.0 / 18.0;
let thickness = (font_metrics.underline_thickness().unwrap_or(1.0) * thickness_factor)
.max(min_thickness); .max(min_thickness);
if style_metric.text_style.decoration().ty == TextDecoration::UNDERLINE { if style_metric.text_style.decoration().ty == TextDecoration::UNDERLINE {
let y = line_baseline + font_metrics.underline_position().unwrap_or(thickness); // Same gap from baseline to underline as in Chromium
// (see https://source.chromium.org/chromium/chromium/src/+/main:ui/gfx/render_text.cc
let gap_scaling = raw_font_size * 1.0 / 9.0;
let y = line_baseline + gap_scaling;
max_underline_thickness = max_underline_thickness.max(thickness); max_underline_thickness = max_underline_thickness.max(thickness);
underline_y = Some(y); underline_y = Some(y);
} }