Skip to content

Commit

Permalink
[monaco] removed overriding dark-plus theming
Browse files Browse the repository at this point in the history
Fixes eclipse-theia#5188

Signed-off-by: Sven Efftinge <[email protected]>
  • Loading branch information
svenefftinge committed May 21, 2019
1 parent c52b47d commit f39185a
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 40 deletions.
4 changes: 2 additions & 2 deletions packages/core/src/browser/theming.ts
Original file line number Diff line number Diff line change
Expand Up @@ -201,7 +201,7 @@ export class BuiltinThemeProvider {
id: 'dark',
label: 'Dark Theme',
description: 'Bright fonts on dark backgrounds.',
editorTheme: 'vs-dark',
editorTheme: 'dark-plus', // loaded in /packages/monaco/src/browser/textmate/monaco-theme-registry.ts
activate() {
BuiltinThemeProvider.darkCss.use();
},
Expand All @@ -214,7 +214,7 @@ export class BuiltinThemeProvider {
id: 'light',
label: 'Light Theme',
description: 'Dark fonts on light backgrounds.',
editorTheme: 'vs',
editorTheme: 'light-plus', // loaded in /packages/monaco/src/browser/textmate/monaco-theme-registry.ts
activate() {
BuiltinThemeProvider.lightCss.use();
},
Expand Down

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,6 @@
import { interfaces } from 'inversify';
import { FrontendApplicationContribution, isBasicWasmSupported } from '@theia/core/lib/browser';
import { bindContributionProvider } from '@theia/core';
import { ThemeService } from '@theia/core/lib/browser/theming';
import { BuiltinTextmateThemeProvider } from './monaco-textmate-builtin-theme-provider';
import { TextmateRegistry } from './textmate-registry';
import { LanguageGrammarDefinitionContribution } from './textmate-contribution';
import { MonacoTextmateService, OnigasmPromise } from './monaco-textmate-service';
Expand Down Expand Up @@ -68,7 +66,4 @@ export default (bind: interfaces.Bind, unbind: interfaces.Unbind, isBound: inter
bindContributionProvider(bind, LanguageGrammarDefinitionContribution);
bind(TextmateRegistry).toSelf().inSingletonScope();
bind(MonacoThemeRegistry).toDynamicValue(() => MonacoThemeRegistry.SINGLETON).inSingletonScope();

const themeService = ThemeService.get();
themeService.register(...BuiltinTextmateThemeProvider.theiaTextmateThemes);
};

0 comments on commit f39185a

Please sign in to comment.