Use syntax as merged name instead of mergedSyntax

This lets us retain the existing uses of syntax, and fixes colors that were being assigned incorrectly.
This commit is contained in:
Nate Butler 2023-02-26 01:20:44 -05:00
parent 23c967418a
commit 4f307c7601

View file

@ -38,7 +38,7 @@ export default function editor(colorScheme: ColorScheme) {
} }
} }
const syntax: Syntax = { const defaultSyntax: Syntax = {
primary: { primary: {
color: colorScheme.ramps.neutral(1).hex(), color: colorScheme.ramps.neutral(1).hex(),
weight: fontWeights.normal, weight: fontWeights.normal,
@ -134,10 +134,10 @@ export default function editor(colorScheme: ColorScheme) {
function createSyntax(colorScheme: ColorScheme): Syntax { function createSyntax(colorScheme: ColorScheme): Syntax {
if (!colorScheme.syntax) { if (!colorScheme.syntax) {
return syntax return defaultSyntax
} }
return deepmerge<Syntax, Partial<ThemeSyntax>>(syntax, colorScheme.syntax, { return deepmerge<Syntax, Partial<ThemeSyntax>>(defaultSyntax, colorScheme.syntax, {
arrayMerge: (destinationArray, sourceArray) => [ arrayMerge: (destinationArray, sourceArray) => [
...destinationArray, ...destinationArray,
...sourceArray, ...sourceArray,
@ -145,7 +145,7 @@ export default function editor(colorScheme: ColorScheme) {
}); });
} }
const mergedSyntax = createSyntax(colorScheme) const syntax = createSyntax(colorScheme)
return { return {
textColor: syntax.primary.color, textColor: syntax.primary.color,
@ -303,6 +303,6 @@ export default function editor(colorScheme: ColorScheme) {
color: borderColor(layer), color: borderColor(layer),
}, },
}, },
syntax: mergedSyntax, syntax,
} }
} }