fix(sketch): update based on theme changes (#4274)

* fix(sketch): update based on theme changes - Only sync color styles for themes - Make sure yellow-30 is used instead of yellow-20 for yellow in colors * Update packages/sketch/src/sharedStyles/themes.js Co-Authored-By: Abbey Hart <[email protected]>
carbon-design-system · Oct 22, 2019 · f37bd54 · f37bd54
1 parent 5b42ce2
commit f37bd54
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/packages/sketch/src/sharedStyles/colors.js b/packages/sketch/src/sharedStyles/colors.js
@@ -35,7 +35,7 @@ export function syncColorStyles(document) {
     ['black', black['100']],
     ['white', white['0']],
     ['orange', orange['40']],
-    ['yellow', yellow['20']],
+    ['yellow', yellow['30']],
   ].map(([name, value]) => {
     return syncColorStyle(document, formatSharedStyleName(name), value);
   });

diff --git a/packages/sketch/src/sharedStyles/themes.js b/packages/sketch/src/sharedStyles/themes.js
@@ -11,10 +11,13 @@ import {
   g90,
   g100,
   formatTokenName,
+  tokens,
   unstable__meta as meta,
 } from '@carbon/themes';
 import { syncColorStyle } from '../tools/sharedStyles';
 
+const { colors } = tokens;
+
 /**
  * Sync theme color shared styles to the given document and return the result
  * @param {Document} document
@@ -31,7 +34,7 @@ export function syncThemeColorStyles(document) {
   const sharedStyles = Object.keys(themes).flatMap(theme => {
     return Object.keys(themes[theme])
       .filter(token => {
-        return !meta.deprecated.includes(token);
+        return colors.includes(token) && !meta.deprecated.includes(token);
       })
       .map(token => {
         const { type } = meta.colors.find(group => {