Skip to content

Commit

Permalink
Use gray and white as the default dialog theme (#637)
Browse files Browse the repository at this point in the history
I think this has a few benefits:

1. Most terminal themes generally don't change or pick similar colors
   for their black and white colors, so it's more likely to be consistent
    across terminals/themes.
2. They don't have the potential color-blind issues that other color options
   may have.
3. We won't need additional changes for no color mode.
  • Loading branch information
st0012 committed Feb 25, 2024
1 parent 106a529 commit 6579a0a
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions lib/reline/face.rb
Original file line number Diff line number Diff line change
Expand Up @@ -186,9 +186,9 @@ def self.load_initial_configs
conf.define :scrollbar, style: :reset
end
config(:completion_dialog) do |conf|
conf.define :default, foreground: :white, background: :cyan
conf.define :enhanced, foreground: :white, background: :magenta
conf.define :scrollbar, foreground: :white, background: :cyan
conf.define :default, foreground: :bright_white, background: :gray
conf.define :enhanced, foreground: :black, background: :white
conf.define :scrollbar, foreground: :white, background: :gray
end
end

Expand Down

0 comments on commit 6579a0a

Please sign in to comment.