Fix 367

src/SixLabors.Fonts/TextLayout.cs

@@ -1104,6 +1104,12 @@ private static TextBox BreakLines(
                                 textLine = split;
                                 lineAdvance = split.ScaledLineAdvance;
                             }
+                            else if (textLine.Count > 0)
+                            {
+                                textLines.Add(textLine.Finalize());
+                                textLine = new();
+                                lineAdvance = 0;
+                            }
                         }
                         else if (lastLineBreak.PositionWrap < codePointIndex && !CodePoint.IsWhiteSpace(codePoint))
                         {

tests/SixLabors.Fonts.Tests/Issues/Issues_367.cs

@@ -0,0 +1,29 @@
+// Copyright (c) Six Labors.
+// Licensed under the Six Labors Split License.
+
+namespace SixLabors.Fonts.Tests.Issues;
+public class Issues_367
+{
+    [Fact]
+    public void ShouldMatchBrowserBreak()
+    {
+        Font font = new FontCollection().Add(TestFonts.CourierPrimeFile).CreateFont(12);
+
+        TextOptions options = new(font)
+        {
+            Dpi = 96f // 1in = 96px
+        };
+
+        const float wrappingLengthInInches = 3.875f;
+        options.WrappingLength = wrappingLengthInInches * options.Dpi;
+
+        const string text = "Leer, but lonesome has fussin' change a faith. Themself seen and four trample.";
+        int lineCount = TextMeasurer.CountLines(text, options);
+
+        Assert.Equal(3, lineCount);
+
+        FontRectangle advance = TextMeasurer.MeasureAdvance(text, options);
+        Assert.Equal(365, advance.Width);
+        Assert.Equal(48, advance.Height);
+    }
+}

tests/SixLabors.Fonts.Tests/TestFonts.cs

@@ -253,6 +253,8 @@ public static class TestFonts
 
     public static string PermanentMarkerRegularWoff2File => GetFullPath("PermanentMarker-Regular.woff2");
 
+    public static string CourierPrimeFile => GetFullPath("courier-prime.woff2");
+
     public static Stream TwemojiMozillaData() => OpenStream(TwemojiMozillaFile);
 
     public static Stream SegoeuiEmojiData() => OpenStream(SegoeuiEmojiFile);

tests/SixLabors.Fonts.Tests/TextLayoutTests.cs

@@ -493,7 +493,7 @@ public void CountLinesWithSpan()
     [Theory]
     [InlineData("This is a long and Honorificabilitudinitatibus califragilisticexpialidocious", 25, 7)]
     [InlineData("This is a long and Honorificabilitudinitatibus califragilisticexpialidocious", 50, 7)]
-    [InlineData("This is a long and Honorificabilitudinitatibus califragilisticexpialidocious", 100, 6)]
+    [InlineData("This is a long and Honorificabilitudinitatibus califragilisticexpialidocious", 100, 7)]
     [InlineData("This is a long and Honorificabilitudinitatibus califragilisticexpialidocious", 200, 6)]
     public void CountLinesWrappingLength(string text, int wrappingLength, int usedLines)
     {