Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Experiment with back- foreground colors #23

Closed
alloy opened this issue Jun 14, 2011 · 7 comments
Closed

Experiment with back- foreground colors #23

alloy opened this issue Jun 14, 2011 · 7 comments

Comments

@alloy
Copy link
Owner

alloy commented Jun 14, 2011

It is hard on the eyes with a white background in the file browser if you have a dark color scheme.

Option 1: Use the same as the text view

Cons:

  • the icons may blend in with the background color
  • I think (maybe) that the file browser should be visually distinct from the text view
  • background/foreground colors that look good for the text view may look bad for the file browser (this is basically the same as the first point)

If you want to try it out then you can get the background/foreground colors from -[NSWindowController setDefaultColorsBackground:foreground:] and simply pass those on to the file browser. I'd like to try it out too if you write a patch.

Option 2: Provide two presets

One for a dark background and one for a light background.

@alloy
Copy link
Owner Author

alloy commented Jun 16, 2011

A dirty patch that shows roughly what it would work like: https://gist.github.com/1030607

@alloy
Copy link
Owner Author

alloy commented Jun 17, 2011

FYI I personally prefer the normal style, i.e. white background, even though I have a dark vim view background.

@b4winckler
Copy link
Collaborator

Thanks for the patch. My initial reaction is ... argh! It looks terrible. I think the "cons" pretty much all apply. Even having only white backgrounds with black text is better than this.

I'm not sure how the two presets would work either. What colors should be used for dark color schemes?

@alloy
Copy link
Owner Author

alloy commented Jun 17, 2011

Hehehe, yeah I had the same reaction, hence I didn’t take the time to create a nice patch.

I could ask our designer to think about the color scheme for a dark scheme, but as of right now I’m unsure if there’s one good solution to this. Although I dislike making things too configurable, maybe we should do it in this case and see what the community comes up with?

@b4winckler
Copy link
Collaborator

I also dislike lots of configuration options. If there were to be an option to set the background then it really needs to be accessible from Vim script in order to be useful and I think this is overkill at this stage. I vote for just leaving things as they are for now.

@alloy
Copy link
Owner Author

alloy commented Jun 17, 2011

Agreed.

@alloy alloy closed this as completed Jun 17, 2011
@jordwalke
Copy link

I feel this feature would be valuable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants