Skip to content

Commit

Permalink
Added new color variables for warning alert (as in scm widget)
Browse files Browse the repository at this point in the history
so one is not forced to use 'inputValidation.warningBackground'
as foreground and 'editorWarning.foreground' as background and can
overwrite the vars for the warning field now without changing
colors for completely different elements.

Signed-off-by: Jan Bicker <jan.bicker@typefox.io>
  • Loading branch information
jbicker committed Jan 15, 2020
1 parent 3acbec2 commit 183ba3e
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 2 deletions.
16 changes: 16 additions & 0 deletions packages/core/src/browser/common-frontend-contribution.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1541,6 +1541,22 @@ export class CommonFrontendContribution implements FrontendApplicationContributi
hc: 'editorGutter.addedBackground'
}, description: 'Background color of success widgets (like alerts or notifications).'
},
{
id: 'warningBackground',
defaults: {
dark: 'editorWarning.foreground',
light: 'editorWarning.foreground',
hc: 'editorWarning.border'
}, description: 'Background color of warning widgets (like alerts or notifications).'
},
{
id: 'warningForeground',
defaults: {
dark: 'inputValidation.warningBackground',
light: 'inputValidation.warningBackground',
hc: 'inputValidation.warningBackground'
}, description: 'Foreground color of warning widgets (like alerts or notifications).'
},
// Statusbar
{
id: 'statusBar.offlineBackground',
Expand Down
4 changes: 2 additions & 2 deletions packages/core/src/browser/style/alert-messages.css
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,8 @@

/* warning message */
.theia-warning-alert {
background-color: var(--theia-editorWarning-foreground);
color: var(--theia-inputValidation-warningBackground);
background-color: var(--theia-warningBackground);
color: var(--theia-warningForeground);
padding: 10px;
}

Expand Down

0 comments on commit 183ba3e

Please sign in to comment.