Correctly implement (Partially)Resolvable

Siteimprove · Nov 23, 2023 · 41d3074 · 41d3074
1 parent b8c7f57
commit 41d3074
Show file tree

Hide file tree

Showing 6 changed files with 31 additions and 9 deletions.
diff --git a/.changeset/gentle-berries-decide.md b/.changeset/gentle-berries-decide.md
@@ -0,0 +1,5 @@
+---
+"@siteimprove/alfa-css": patch
+---
+
+**Added:** all `Value` subtype now correctly implement the `(Partially)Resovable` interfaces.
diff --git a/packages/alfa-css/src/value/image/gradient/linear/corner.ts b/packages/alfa-css/src/value/image/gradient/linear/corner.ts
@@ -3,6 +3,7 @@ import { Parser } from "@siteimprove/alfa-parser";
 
 import { type Parser as CSSParser, Token } from "../../../../syntax";
 
+import { Resolvable } from "../../../resolvable";
 import { Value } from "../../../value";
 
 import { Position } from "./position";
@@ -12,7 +13,10 @@ const { map, either, pair, option, right } = Parser;
 /**
  * @internal
  */
-export class Corner extends Value<"corner", false> {
+export class Corner
+  extends Value<"corner", false>
+  implements Resolvable<Corner.Canonical, Corner.Resolver>
+{
   public static of(
     vertical: Position.Vertical,
     horizontal: Position.Horizontal,

diff --git a/packages/alfa-css/src/value/image/gradient/linear/side.ts b/packages/alfa-css/src/value/image/gradient/linear/side.ts
@@ -3,6 +3,7 @@ import { Parser } from "@siteimprove/alfa-parser";
 
 import { Token } from "../../../../syntax";
 
+import { Resolvable } from "../../../resolvable";
 import { Value } from "../../../value";
 
 import { Position } from "./position";
@@ -12,7 +13,10 @@ const { map, option, right } = Parser;
 /**
  * @internal
  */
-export class Side extends Value<"side", false> {
+export class Side
+  extends Value<"side", false>
+  implements Resolvable<Side.Canonical, Side.Resolver>
+{
   public static of(side: Position.Vertical | Position.Horizontal): Side {
     return new Side(side);
   }

diff --git a/packages/alfa-css/src/value/image/gradient/radial/circle.ts b/packages/alfa-css/src/value/image/gradient/radial/circle.ts
@@ -5,17 +5,18 @@ import { Parser as CSSParser, Token } from "../../../../syntax";
 
 import { Keyword } from "../../../keyword";
 import { Length } from "../../../numeric";
+import { Resolvable } from "../../../resolvable";
 import { Value } from "../../../value";
 
 /**
  * {@link https://drafts.csswg.org/css-images/#valdef-ending-shape-circle}
  *
  * @internal
  */
-export class Circle<R extends Length = Length> extends Value<
-  "circle",
-  Value.HasCalculation<[R]>
-> {
+export class Circle<R extends Length = Length>
+  extends Value<"circle", Value.HasCalculation<[R]>>
+  implements Resolvable<Circle.Canonical, Circle.Resolver>
+{
   public static of<R extends Length>(radius: R): Circle<R> {
     return new Circle(radius);
   }

diff --git a/packages/alfa-css/src/value/image/gradient/radial/extent.ts b/packages/alfa-css/src/value/image/gradient/radial/extent.ts
@@ -5,14 +5,18 @@ import { Err, Result } from "@siteimprove/alfa-result";
 import { Parser as CSSParser, Token } from "../../../../syntax";
 
 import { Keyword } from "../../../keyword";
+import { Resolvable } from "../../../resolvable";
 import { Value } from "../../../value";
 
 const { map } = Parser;
 
 /**
  * @internal
  */
-export class Extent extends Value<"extent", false> {
+export class Extent
+  extends Value<"extent", false>
+  implements Resolvable<Extent.Canonical, Extent.Resolver>
+{
   public static of(
     shape: Extent.Shape = Extent.Shape.Circle,
     size: Extent.Size = Extent.Size.FarthestCorner,

diff --git a/packages/alfa-css/src/value/shape/rectangle.ts b/packages/alfa-css/src/value/shape/rectangle.ts
@@ -5,6 +5,7 @@ import { Comma, Function, type Parser as CSSParser, Token } from "../../syntax";
 
 import { Keyword } from "../keyword";
 import { Length } from "../numeric";
+import { Resolvable } from "../resolvable";
 import { Value } from "../value";
 
 import { BasicShape } from "./basic-shape";
@@ -18,8 +19,11 @@ const { either, map, option, separatedList } = Parser;
  * @deprecated Deprecated as of CSS Masking Module Level 1
  */
 export class Rectangle<
-  O extends Length | Rectangle.Auto = Length | Rectangle.Auto,
-> extends BasicShape<"rectangle", Value.HasCalculation<[O, O, O, O]>> {
+    O extends Length | Rectangle.Auto = Length | Rectangle.Auto,
+  >
+  extends BasicShape<"rectangle", Value.HasCalculation<[O, O, O, O]>>
+  implements Resolvable<Rectangle.Canonical, Rectangle.Resolver>
+{
   public static of<O extends Length | Rectangle.Auto = Length | Rectangle.Auto>(
     top: O,
     right: O,