From eb4aee0d8bbd89d794a705d7ec8ac97149f2e7d0 Mon Sep 17 00:00:00 2001
From: Benedikt Stebner <Gillibald@users.noreply.github.com>
Date: Wed, 22 Nov 2023 00:29:10 +0100
Subject: [PATCH] [WIP]Fix overhang leading calculation for negative values
 (#13672)

* [macOS] Fix overhand leading calculation

* Adjust TextLineMetrics

* Update expected result
---
 .../Media/TextFormatting/TextLineImpl.cs        |   6 +++---
 .../RestrictedHeight_VerticalAlign.expected.png | Bin 528 -> 529 bytes
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/Avalonia.Base/Media/TextFormatting/TextLineImpl.cs b/src/Avalonia.Base/Media/TextFormatting/TextLineImpl.cs
index b4534fd274..efaab511ce 100644
--- a/src/Avalonia.Base/Media/TextFormatting/TextLineImpl.cs
+++ b/src/Avalonia.Base/Media/TextFormatting/TextLineImpl.cs
@@ -1325,12 +1325,12 @@ namespace Avalonia.Media.TextFormatting
                 }
             }
 
-            //The width of overhanging pixels ath the bottom
+            //The width of overhanging pixels at the bottom
             var overhangAfter = Math.Max(0, bounds.Bottom - height);
             //The width of overhanging pixels at the origin
-            var overhangLeading = bounds.Left;
+            var overhangLeading = Math.Abs(Math.Min(bounds.Left, 0));
             //The width of overhanging pixels at the end
-            var overhangTrailing = Math.Max(0, bounds.Width - widthIncludingWhitespace);
+            var overhangTrailing = Math.Max(0, bounds.Right - widthIncludingWhitespace);
             var hasOverflowed = width > _paragraphWidth;
 
             if (!double.IsNaN(lineHeight) && !MathUtilities.IsZero(lineHeight))
diff --git a/tests/TestFiles/Skia/Controls/TextBlock/RestrictedHeight_VerticalAlign.expected.png b/tests/TestFiles/Skia/Controls/TextBlock/RestrictedHeight_VerticalAlign.expected.png
index c8320cdb97c182922987f97e07a33f1c68eff45e..30e2ca86621188f40d0080454681b72b4fcf83c7 100644
GIT binary patch
delta 487
zcmbQhGLdD1N&N{=7srr_IdAV620eBVU^wvb71N>J6ME9`=s&RhE|a?C&!PB2kBXUk
z1|n^TGQ#C5Q*+Cw3Ps9!25<hhCdHD80Ub1~E(_jv(?adds)G0pd*&~d?pvDtgxh?g
zL7u|J{%h<Ex+3+D4zo_#!ltRa`B1uVeP8#l&k7!MmCZE%oax*Yal7}cgkZh$qw}}B
zI{#jcwb9(Ss`%fNZ+}&)d$#s>?AG|JxBp!6nI2R7_|#Qu+w}`2G~;!@n>J2L{_3!=
zD0HjU{ztVe*LqGY1@m{#@tiJV7aR0L+tB4m@$2RtO3rhpulE)CC#H7ycgK(I34fC7
zKW&{T9ieu&^hvhz{2u|?8xP4VtxrGObG2FVN%pCvwdd@V^1>xvPl{7|^nH2h{kw5B
zUboiQ_}+Hu-ye4Tul{AdnfkUM8%kELx_a`>!KknMon`g~$%l(&chx+d^;dao@zz)M
z>q{@`#_!x>RpR;ZaOPtRxq1Avx7O5bd9x-UDPigHDhcH>^?Tc^YxlaY+4($T*}=u_
zYaB`!9(fscvZvHAFX5}^<`1*PKgv|q>{{~SWl-Vj$N%*i(Sx0#;pz|mX@*<4Ha_X*
P1xb3k`njxgN@xNA@onXJ

delta 486
zcmbQpGJ$1+N&Rt87srr_IdAV67R`1LVS8|YrikWTqu9QO(Kn8NEPQe4nb^KAuj%g!
z7FImbn7DPp(e4}XX9p_JV7J-pn(wH=$bb$Wthzg8ty$kxv+y0Z5!bF?5<Py&u;`Gl
zR<G=e2i~t`8^pTK|JWe(JgR-sHJ69Qx%KzBepfK~ygDi^TKD$kri$8Q)gGT_JkhiK
z?>phUczD<PS661<k>hjOlQR8@dzaltUzz;36`wzwCl~#m^Ecz?{nyX0Byb;j-TUXP
z@y@F|&nC9`N*2x+U2p!$y#Msf63>U1Ra^HbIZwL3w){~SujSnRXGQ-_Q@g)gMV~wW
zafJTQ+x2&Nbtb<*{bkbrvvWT#JhG_Oeo0<`lyzKw+ll+LU8Cp5PRiHryMCy6(tG#!
zwI%(Z!;j{kzJKsp)t{}^_dgrOuKn}!<I72hmU~Z*o_otS>-4<x6aQ9!PF(uY>%@H<
z@xLBx+osvR7gM_rbWT{io@R|juFUbjpW`>j*W25b6<U0>i&wq(cZSW}h;LugD^FJx
z_iw+sxP8ryin^)wHD7jK>kDs{j*wX$Te8~bCe)&%Yy1B1V?hsf28OW5qBdWT{t!6p
Q(GQaJboFyt=akR{02U|p%K!iX