Deal with the processing attribute

Co-authored-by: Ricky Atkins <[email protected]>

src/designspace.rs

@@ -23,8 +23,8 @@ pub struct DesignSpaceDocument {
     #[serde(with = "serde_impls::axes", skip_serializing_if = "Vec::is_empty")]
     pub axes: Vec<Axis>,
     /// One or more rules.
-    #[serde(default, with = "serde_impls::rules", skip_serializing_if = "Vec::is_empty")]
-    pub rules: Vec<Rule>,
+    #[serde(default, skip_serializing_if = "Rules::is_empty")]
+    pub rules: Rules,
     /// One or more sources.
     #[serde(with = "serde_impls::sources", skip_serializing_if = "Vec::is_empty")]
     pub sources: Vec<Source>,
@@ -81,11 +81,32 @@ pub struct AxisMapping {
     pub output: f32,
 }
 
+/// Describes the substitution rules of the Designspace.
+#[derive(Clone, Debug, Default, PartialEq, Serialize, Deserialize)]
+pub struct Rules {
+    /// Name of the rule.
+    #[serde(rename = "@processing")]
+    pub processing: RuleProcessing,
+    /// The rules.
+    #[serde(default, rename = "rule")]
+    pub rules: Vec<Rule>,
+}
+
+/// Indicates whether substitution rules should be applied before or after other glyph substitution features.
+#[derive(Clone, Debug, Default, PartialEq, Serialize, Deserialize)]
+#[serde(rename_all = "lowercase")]
+pub enum RuleProcessing {
+    /// Apply before other substitution features.
+    #[default]
+    First,
+    /// Apply after other substitution features.
+    Last,
+}
+
 /// Describes a single set of substitution rules.
 ///
-/// Does not support standalone <condition> elements outside a <conditionset>.
+/// Does not support standalone `<condition>` elements outside a `<conditionset>`.
 #[derive(Clone, Debug, Default, PartialEq, Serialize, Deserialize)]
-#[serde(rename = "rule")]
 pub struct Rule {
     /// Name of the rule.
     #[serde(rename = "@name")]
@@ -236,6 +257,13 @@ impl DesignSpaceDocument {
     }
 }
 
+impl Rules {
+    /// Returns `true` if there are no rules.
+    fn is_empty(&self) -> bool {
+        self.rules.is_empty()
+    }
+}
+
 mod serde_impls {
     /// Produces a self-contained module to (de)serialise an XML list of a given type
     ///
@@ -326,7 +354,6 @@ mod serde_impls {
     serde_from_field!(instances, instance, crate::designspace::Instance);
     serde_from_field!(axes, axis, crate::designspace::Axis);
     serde_from_field!(sources, source, crate::designspace::Source);
-    serde_from_field!(rules, rule, crate::designspace::Rule);
 }
 
 #[cfg(test)]
@@ -461,37 +488,46 @@ mod tests {
         // Then
         assert_eq!(
             &ds_after.rules,
-            &[
-                Rule {
-                    name: Some("fold_I_serifs".into()),
-                    condition_sets: vec![ConditionSet {
-                        conditions: vec![Condition {
-                            name: "width".into(),
-                            minimum: Some(0.0),
-                            maximum: Some(328.0),
-                        }],
-                    }],
-                    substitutions: vec![Substitution { name: "I".into(), with: "I.narrow".into() }],
-                },
-                Rule {
-                    name: Some("fold_S_terminals".into()),
-                    condition_sets: vec![ConditionSet {
-                        conditions: vec![
-                            Condition {
+            &Rules {
+                processing: RuleProcessing::Last,
+                rules: vec![
+                    Rule {
+                        name: Some("fold_I_serifs".into()),
+                        condition_sets: vec![ConditionSet {
+                            conditions: vec![Condition {
                                 name: "width".into(),
                                 minimum: Some(0.0),
-                                maximum: Some(1000.0),
-                            },
-                            Condition {
-                                name: "weight".into(),
-                                minimum: Some(0.0),
-                                maximum: Some(500.0),
-                            },
-                        ],
-                    }],
-                    substitutions: vec![Substitution { name: "S".into(), with: "S.closed".into() }],
-                },
-            ]
+                                maximum: Some(328.0),
+                            }],
+                        }],
+                        substitutions: vec![Substitution {
+                            name: "I".into(),
+                            with: "I.narrow".into()
+                        }],
+                    },
+                    Rule {
+                        name: Some("fold_S_terminals".into()),
+                        condition_sets: vec![ConditionSet {
+                            conditions: vec![
+                                Condition {
+                                    name: "width".into(),
+                                    minimum: Some(0.0),
+                                    maximum: Some(1000.0),
+                                },
+                                Condition {
+                                    name: "weight".into(),
+                                    minimum: Some(0.0),
+                                    maximum: Some(500.0),
+                                },
+                            ],
+                        }],
+                        substitutions: vec![Substitution {
+                            name: "S".into(),
+                            with: "S.closed".into()
+                        }],
+                    },
+                ]
+            }
         );
         assert_eq!(ds_initial, ds_after);
     }

testdata/MutatorSans.designspace

@@ -4,7 +4,7 @@
     <axis tag="wdth" name="width" minimum="0" maximum="1000" default="0"/>
     <axis tag="wght" name="weight" minimum="0" maximum="1000" default="0"/>
   </axes>
-  <rules>
+  <rules processing="last">
     <rule name="fold_I_serifs">
       <conditionset>
         <condition name="width" minimum="0" maximum="328"/>