Merge pull request #356 from SixLabors/js/v1-fix-355

Fix 355
SixLabors · Sep 15, 2023 · be8883c · be8883c
2 parents dc3e081 + b10de71
commit be8883c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/.github/workflows/build-and-test.yml b/.github/workflows/build-and-test.yml
@@ -9,7 +9,7 @@ on:
   pull_request:
     branches:
       - main
-
+      - release/*
 jobs:
   Build:
     strategy:

diff --git a/src/SixLabors.Fonts/TextLayout.cs b/src/SixLabors.Fonts/TextLayout.cs
@@ -828,6 +828,11 @@ private static void SubstituteBidiMirrors(FontMetrics fontMetrics, GlyphSubstitu
 
             // TODO: This only replaces certain glyphs. We should investigate the specification further.
             // https://www.unicode.org/reports/tr50/#vertical_alternates
+            if (collection.TextOptions.LayoutMode.IsHorizontal())
+            {
+                return;
+            }
+
             for (int i = 0; i < collection.Count; i++)
             {
                 GlyphShapingData data = collection[i];