Skip to content

Commit

Permalink
Adjust AlertMessage font-color for bright themes
Browse files Browse the repository at this point in the history
Before the change, the color was determined through variables defined
through the dark and bright variables.css but this should not apply
for `AlertMessage`s since we control the background color and want
the proper contrast. If a white theme was used before, we displayed
black font which was not the intended or wanted styling.

Signed-off-by: Vincent Fugnitto <[email protected]>
  • Loading branch information
vince-fugnitto authored and elaihau committed Dec 10, 2018
1 parent ec57ef5 commit bd40231
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions packages/core/src/browser/style/alert-messages.css
Original file line number Diff line number Diff line change
Expand Up @@ -27,14 +27,14 @@
/* information message */
.theia-info-alert {
background-color: var(--theia-brand-color0);
color: var(--theia-ui-font-color0);
color: #ffffff;
padding: 10px;
}

/* success message */
.theia-success-alert {
background-color: var(--theia-success-color0);
color: var(--theia-ui-font-color0);
color: #ffffff;
padding: 10px;
}

Expand All @@ -48,6 +48,6 @@
/* error message */
.theia-error-alert {
background-color: var(--theia-error-color0);
color: var(--theia-ui-font-color0);
color: #ffffff;
padding: 10px;
}

0 comments on commit bd40231

Please sign in to comment.