Mark keyframe easing values as optional in schema (#67)

* Mark keyframe easing values as optional in schema Docs updated to specify when these are required * Add back accidentally deleted section
lottie · Jul 31, 2024 · f8ff9ba · f8ff9ba
1 parent 944f29d
commit f8ff9ba
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 19 deletions.
diff --git a/docs/specs/properties.md b/docs/specs/properties.md
@@ -17,8 +17,11 @@ Their structure depends on whether it's animated or not:
 
 Keyframe arrays MUST be stored in order of strictly ascending `t` frame number. Two keyframes cannot have the same `t` value.
 
-If `h` is present and it's 1, you don't need `i` and `o`, as the property will keep the same value
-until the next keyframe.
+All keyframes MUST have an `i` and `o` value, unless-
+
+* It is the last keyframe in the sequence OR
+* `h` is present and it's 1, as the property will keep the same value until the
+next keyframe.
 
 If the first keyframe occurs after the start of the animation, the initial property value will be from the first keyframe. Similarly if the last keyframe is before the end of the animation, the last keyframe value will be held until the end.
 

diff --git a/schema/properties/base-keyframe.json b/schema/properties/base-keyframe.json
@@ -28,23 +28,6 @@
                     "$ref": "#/$defs/properties/easing-handle"
                 }
             }
-        },
-        {
-            "if": {
-                "oneOf": [
-                    {
-                        "properties": {
-                            "h": { "const": 0  }
-                        }
-                    },
-                    {
-                        "not": {"required": ["h"]}
-                    }
-                ]
-            },
-            "then": {
-                "required": ["i", "o"]
-            }
         }
     ],
     "required": ["t"]