From 556886e0b1eb5ab77a513c6fcc1cec1e54f46201 Mon Sep 17 00:00:00 2001
From: John McWilliams <37010132+jmcwilliams403@users.noreply.github.com>
Date: Wed, 1 Nov 2023 13:34:18 -0400
Subject: [PATCH] Make `U+1EFE` follow `cv24`. (#2070)

---
 changes/27.3.4.md                        |  1 +
 font-src/glyphs/letter/latin/lower-y.ptl |  2 +-
 params/variants.toml                     | 18 ++++++++++++------
 3 files changed, 14 insertions(+), 7 deletions(-)

diff --git a/changes/27.3.4.md b/changes/27.3.4.md
index 54db0fd22..4f684237a 100644
--- a/changes/27.3.4.md
+++ b/changes/27.3.4.md
@@ -1 +1,2 @@
 * Disunify anonymous untagged variant selectors for Cyrillic Capital Yeri/Yery for consistency in style-driven configurations.
+* Make LATIN CAPITAL LETTER Y WITH LOOP (`U+1EFE`) follow variants of capital `Y` (`cv24`) for a more balanced slab-italic form like that of Cyrillic Capital U.
diff --git a/font-src/glyphs/letter/latin/lower-y.ptl b/font-src/glyphs/letter/latin/lower-y.ptl
index 70dcd00cc..ff5756028 100644
--- a/font-src/glyphs/letter/latin/lower-y.ptl
+++ b/font-src/glyphs/letter/latin/lower-y.ptl
@@ -430,8 +430,8 @@ glyph-block Letter-Latin-Lower-Y : begin
 	select-variant 'yHookTop' 0x1B4
 	select-variant 'cyrl/U' 0x423 (shapeFrom -- 'yCap')
 
+	select-variant 'YLoop' 0x1EFE (shapeFrom -- 'yCap')
 	select-variant 'yLoop' 0x1EFF (shapeFrom -- 'y')
-	select-variant 'YLoop' 0x1EFE (follow -- 'yLoop') (shapeFrom -- 'yCap')
 
 	select-variant 'grek/lambda' 0x3BB
 	select-variant 'lambdaSlash' 0x19B (follow -- 'grek/lambda')
diff --git a/params/variants.toml b/params/variants.toml
index ae2da4345..f49cd8fca 100644
--- a/params/variants.toml
+++ b/params/variants.toml
@@ -1324,44 +1324,50 @@ next = "serifs"
 rank = 1
 descriptionAffix = "straight shape"
 selectorAffix.Y = "straight"
-selectorAffix."grek/upsilonHookedSymbolShape" = "straight"
 selectorAffix."Y/sansSerif" = "straight"
+selectorAffix.YLoop = "straightLoop"
+selectorAffix."grek/upsilonHookedSymbolShape" = "straight"
 
 [prime.capital-y.variants-buildup.stages.body.curly]
 rank = 2
 descriptionAffix = "curly shape"
 selectorAffix.Y = "curly"
-selectorAffix."grek/upsilonHookedSymbolShape" = "straight"
 selectorAffix."Y/sansSerif" = "curly"
+selectorAffix.YLoop = "curlyLoop"
+selectorAffix."grek/upsilonHookedSymbolShape" = "straight"
 
 [prime.capital-y.variants-buildup.stages.serifs.serifless]
 rank = 1
 descriptionAffix = "serifs"
 descriptionJoiner = "without"
 selectorAffix.Y = "serifless"
-selectorAffix."grek/upsilonHookedSymbolShape" = "serifless"
 selectorAffix."Y/sansSerif" = "serifless"
+selectorAffix.YLoop = "serifless"
+selectorAffix."grek/upsilonHookedSymbolShape" = "serifless"
 
 [prime.capital-y.variants-buildup.stages.serifs.base-serifed]
 rank = 2
 descriptionAffix = "serifs at bottom"
 selectorAffix.Y = "baseSerifed"
-selectorAffix."grek/upsilonHookedSymbolShape" = "BaseSerifed"
 selectorAffix."Y/sansSerif" = "serifless"
+selectorAffix.YLoop = "serifless"
+selectorAffix."grek/upsilonHookedSymbolShape" = "BaseSerifed"
 
 [prime.capital-y.variants-buildup.stages.serifs.motion-serifed]
 rank = 3
 descriptionAffix = "motion serifs"
 selectorAffix.Y = "motionSerifed"
-selectorAffix."grek/upsilonHookedSymbolShape" = "serifless"
 selectorAffix."Y/sansSerif" = "serifless"
+selectorAffix.YLoop = "motionSerifed"
+selectorAffix."grek/upsilonHookedSymbolShape" = "serifless"
 
 [prime.capital-y.variants-buildup.stages.serifs.serifed]
 rank = 4
 descriptionAffix = "serifs"
 selectorAffix.Y = "serifed"
-selectorAffix."grek/upsilonHookedSymbolShape" = "BaseSerifed"
 selectorAffix."Y/sansSerif" = "serifless"
+selectorAffix.YLoop = "serifed"
+selectorAffix."grek/upsilonHookedSymbolShape" = "BaseSerifed"