<div dir="ltr">Hi Skef,<div><br></div><div>I'm implementing the ConditionValue proposal, and have some feedback. This is based on the following document:</div><div><br></div><div>  <a href="https://github.com/adobe-type-tools/opentype-spec-drafts/blob/main/condvalue_spec.md">https://github.com/adobe-type-tools/opentype-spec-drafts/blob/main/condvalue_spec.md</a></div><div><span style="color:rgb(31,35,40);font-family:-apple-system,BlinkMacSystemFont,"Segoe UI","Noto Sans",Helvetica,Arial,sans-serif,"Apple Color Emoji","Segoe UI Emoji";font-size:16px"><br></span></div><div>"""</div><div><span style="color:rgb(31,35,40);font-family:-apple-system,BlinkMacSystemFont,"Segoe UI","Noto Sans",Helvetica,Arial,sans-serif,"Apple Color Emoji","Segoe UI Emoji";font-size:16px">In "FeatureVariations Table", note that if minorVersion is 0 then only Condition Table version 1 can be used. If minorVersion is 1 then Condition Table version 2 can also be used.</span></div><div><span style="color:rgb(31,35,40);font-family:-apple-system,BlinkMacSystemFont,"Segoe UI","Noto Sans",Helvetica,Arial,sans-serif,"Apple Color Emoji","Segoe UI Emoji";font-size:16px">"""</span></div><div><font color="#1f2328" face="-apple-system, BlinkMacSystemFont, Segoe UI, Noto Sans, Helvetica, Arial, sans-serif, Apple Color Emoji, Segoe UI Emoji"><span style="font-size:16px">I highly suggest removing this. The minorVersion is for when new fields are added to FeatureVariations table itself. It should not be relied on for anything else.</span></font></div><div><font color="#1f2328" face="-apple-system, BlinkMacSystemFont, Segoe UI, Noto Sans, Helvetica, Arial, sans-serif, Apple Color Emoji, Segoe UI Emoji"><span style="font-size:16px"><br></span></font></div><div><span style="color:rgb(31,35,40);font-family:-apple-system,BlinkMacSystemFont,"Segoe UI","Noto Sans",Helvetica,Arial,sans-serif,"Apple Color Emoji","Segoe UI Emoji";font-size:16px">"""</span></div><div><span style="color:rgb(31,35,40);font-family:-apple-system,BlinkMacSystemFont,"Segoe UI","Noto Sans",Helvetica,Arial,sans-serif,"Apple Color Emoji","Segoe UI Emoji";font-size:16px">Add new subpart between "Condition Table Format 1: Font Variation Axis Range" and "FeatureTableSubstitution Table" with this content:</span><br></div><div><span style="color:rgb(31,35,40);font-family:-apple-system,BlinkMacSystemFont,"Segoe UI","Noto Sans",Helvetica,Arial,sans-serif,"Apple Color Emoji","Segoe UI Emoji";font-size:16px">"""</span></div><div><span style="color:rgb(31,35,40);font-family:-apple-system,BlinkMacSystemFont,"Segoe UI","Noto Sans",Helvetica,Arial,sans-serif,"Apple Color Emoji","Segoe UI Emoji";font-size:16px">Can we name Format1 simply ConditionAxisRange? That would go better with ConditionValue, ConditionAnd, ConditionOr, and ConditionNegate.</span></div><div><span style="color:rgb(31,35,40);font-family:-apple-system,BlinkMacSystemFont,"Segoe UI","Noto Sans",Helvetica,Arial,sans-serif,"Apple Color Emoji","Segoe UI Emoji";font-size:16px"><br></span></div><div><span style="color:rgb(31,35,40);font-family:-apple-system,BlinkMacSystemFont,"Segoe UI","Noto Sans",Helvetica,Arial,sans-serif,"Apple Color Emoji","Segoe UI Emoji";font-size:16px">"""</span></div><div>On page 166: "Within the GPOS, JSTF, GDEF and BASE tables, delta-set indices are stored in VariationIndex tables."<br>"""</div><div>I don't understand this. What is a VariationIndex table?</div><div><br></div><div>Thanks,</div><div><br></div><div><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature">behdad<br><a href="http://behdad.org/" target="_blank">http://behdad.org/</a></div></div></div></div>