Making it clear we're using default settings.

1 files changed, 7 insertions, 11 deletions

diff --git a/editors/code/src/scopes.ts b/editors/code/src/scopes.ts
index a099696a972..607b8b2dd63 100644
--- a/editors/code/src/scopes.ts
+++ b/editors/code/src/scopes.ts
@@ -85,17 +85,13 @@ function loadThemeFile(themePath: string) {
         }
     }
 }
-function mergeRuleSettings(defaultRule: TextMateRuleSettings, override: TextMateRuleSettings): TextMateRuleSettings {
-    const mergedRule = defaultRule;
-    if (override.background) {
-        mergedRule.background = override.background
-    }
-    if (override.foreground) {
-        mergedRule.foreground = override.foreground
-    }
-    if (override.background) {
-        mergedRule.fontStyle = override.fontStyle
-    }
+function mergeRuleSettings(defaultSetting: TextMateRuleSettings, override: TextMateRuleSettings): TextMateRuleSettings {
+    const mergedRule = defaultSetting;
+
+    mergedRule.background = override.background || defaultSetting.background
+    mergedRule.foreground = override.foreground || defaultSetting.foreground
+    mergedRule.fontStyle = override.fontStyle || defaultSetting.foreground;
+
     return mergedRule;
 }