💬 Change behaviour of namedColor to only allow LWT color names

kierandrewett · kierandrewett · commit dcbb03d3a023 · 2023-12-28T21:47:21.000Z
diff --git a/components/customizableui/BrowserCustomizableAttributePrimitives.sys.mjs b/components/customizableui/BrowserCustomizableAttributePrimitives.sys.mjs
@@ -152,7 +152,9 @@ export const BrowserCustomizableAttributePrimitives = {
 			return `var(--${value})`;
 		}
 
-		return BrowserCustomizableAttributePrimitives.color(attribute)(value);
+		throw new Error(
+			`Attribute '${attribute}' does not use a valid or supported named color, got '${value}'.`
+		);
 	},
 
 	/**

Original file line number	Diff line number	Diff line change
`@@ -152,7 +152,9 @@ export const BrowserCustomizableAttributePrimitives = {`
`152`	`152`	return `var(--${value})`;
`153`	`153`	`}`
`154`	`154`
`155`		`- return BrowserCustomizableAttributePrimitives.color(attribute)(value);`
	`155`	`+ throw new Error(`
	`156`	+ `Attribute '${attribute}' does not use a valid or supported named color, got '${value}'.`
	`157`	`+ );`
`156`	`158`	`},`
`157`	`159`
`158`	`160`	`/**`