Set font variations settings while extracting a glyph by name (#433)

drawBot/context/baseContext.py

@@ -1223,20 +1223,8 @@ def append(self, txt, **kwargs):
                         # The value 0 means kerning is disabled.
                         attributes[AppKit.NSKernAttributeName] = 0
 
-            coreTextFontVariations = dict()
-            if self._fontVariations:
-                existingAxes = variation.getVariationAxesForFont(font)
-                for axis, value in self._fontVariations.items():
-                    if axis in existingAxes:
-                        existinsAxis = existingAxes[axis]
-                        # clip variation value within the min max value
-                        if value < existinsAxis["minValue"]:
-                            value = existinsAxis["minValue"]
-                        if value > existinsAxis["maxValue"]:
-                            value = existinsAxis["maxValue"]
-                        coreTextFontVariations[variation.convertVariationTagToInt(axis)] = value
-                    else:
-                        warnings.warn("variation axis '%s' not available for '%s'" % (axis, self._font))
+            coreTextFontVariations = variation.getFontVariationAttributes(font, self._fontVariations)
+
             fontAttributes = {}
             if coreTextFontFeatures:
                 fontAttributes[CoreText.kCTFontFeatureSettingsAttribute] = coreTextFontFeatures
@@ -1954,6 +1942,10 @@ def appendGlyph(self, *glyphNames):
         # disable calt features, as this seems to be on by default
         # for both the font stored in the nsGlyphInfo as in the replacement character
         fontAttributes = {}
+        coreTextFontVariations = variation.getFontVariationAttributes(font, self._fontVariations)
+        if coreTextFontVariations:
+            fontAttributes[CoreText.NSFontVariationAttribute] = coreTextFontVariations
+
         fontAttributes[CoreText.kCTFontFeatureSettingsAttribute] = [dict(CTFeatureOpenTypeTag="calt", CTFeatureOpenTypeValue=False)]
         fontDescriptor = font.fontDescriptor()
         fontDescriptor = fontDescriptor.fontDescriptorByAddingAttributes_(fontAttributes)

drawBot/context/tools/variation.py

@@ -3,7 +3,7 @@
 
 from fontTools.ttLib import TTFont
 
-from drawBot.misc import memoize
+from drawBot.misc import memoize, warnings
 
 """
 https://developer.apple.com/documentation/coretext/ctfont/font_variation_axis_dictionary_keys?language=objc
@@ -87,3 +87,21 @@ def getNamedInstancesForFont(font):
 
     ft.close()
     return instances
+
+
+def getFontVariationAttributes(font, fontVariations):
+    coreTextFontVariations = dict()
+    if fontVariations:
+        existingAxes = getVariationAxesForFont(font)
+        for axis, value in fontVariations.items():
+            if axis in existingAxes:
+                existinsAxis = existingAxes[axis]
+                # clip variation value within the min max value
+                if value < existinsAxis["minValue"]:
+                    value = existinsAxis["minValue"]
+                if value > existinsAxis["maxValue"]:
+                    value = existinsAxis["maxValue"]
+                coreTextFontVariations[convertVariationTagToInt(axis)] = value
+            else:
+                warnings.warn("variation axis '%s' not available for '%s'" % (axis, font.fontName()))
+    return coreTextFontVariations

tests/drawBotScripts/appendGlyphWithFontVariations.py

@@ -0,0 +1,15 @@
+# https://github.com/typemytype/drawbot/issues/402
+import drawBot
+drawBot.size(640, 160)
+
+fs = drawBot.FormattedString()
+fs.font("Skia")
+fs.fontSize(200)
+fs.fontVariations(wght=1)
+fs += "&"
+fs.appendGlyph("ampersand")
+fs.fontVariations(wght=2)
+fs.appendGlyph("ampersand")
+fs += "&"
+
+drawBot.text(fs, (10, 10))