Skip to content

Commit

Permalink
Replace black and white default colors with reset. (Rigellute#742)
Browse files Browse the repository at this point in the history
Previously, text would be white, which is not visible on bright color
schemes. The color for non-highlighted text now uses the default
foreground color of the users terminal emulator, making it visible in
any theme. The only visual change in dark color schemes is that the
playbar text might go from black to white, depending on the users
terminal emulator (some terminal emulators swap fore- and background
colors on a colored background depending on the background color; this
is IMO the correct behavior anyway).
  • Loading branch information
Jesse-Bakker authored Feb 27, 2021
1 parent c5b3400 commit 7fa2889
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions src/user_config.rs
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ impl Default for Theme {
fn default() -> Self {
Theme {
analysis_bar: Color::LightCyan,
analysis_bar_text: Color::Black,
analysis_bar_text: Color::Reset,
active: Color::Cyan,
banner: Color::LightCyan,
error_border: Color::Red,
Expand All @@ -64,10 +64,10 @@ impl Default for Theme {
playbar_background: Color::Black,
playbar_progress: Color::LightCyan,
playbar_progress_text: Color::LightCyan,
playbar_text: Color::White,
playbar_text: Color::Reset,
selected: Color::LightCyan,
text: Color::White,
header: Color::White,
text: Color::Reset,
header: Color::Reset,
}
}
}
Expand Down

0 comments on commit 7fa2889

Please sign in to comment.