search

atom / one-light-ui

Atom One light UI theme
MIT License
73 stars 62 forks source link

Find & Replace buttons confusing #3

Closed thedaniel closed 9 years ago

thedaniel commented 9 years ago

The case button (Aa) is toggled on, but with the one-light themes active, it reallly looks like it's off.

screen shot 2014-12-08 at 4 51 36 pm

simurai commented 9 years ago

@thedaniel Do you mean that in this case, .* looks "selected" instead of Aa, or that Aa just looks "disabled" (as in you can't click it)?

How about making the selected state have an accent color and also the text inputs white instead of dark? Something like:

screen shot 2014-12-09 at 11 37 22 am

It's kinda close to OS X Yosemite, but should also make it more clear (especially if you're on Yosemite).

btw. is your screenshot from a newer version where all the other buttons got removed?

thedaniel commented 9 years ago

I mean the first thing you said, that .* looks selected and Aa looks not-selected.

Which other buttons do you mean? This is from a build of master as of a day or two ago...

simurai commented 9 years ago

Which other buttons do you mean?

Ahh.. Never mind, sorry. I had the "find in current buffer" open.

Ok, it's now changed to this:

screen shot 2014-12-09 at 4 41 12 pm