Closed mshron closed 5 years ago
Glad you like it! Yes, I realized after releasing that I'd only tested on my dark terminals! Oops! πIt'd be great to have your help on this, and your suggested method is reasonable. Another option would be to variabilize the colors used here, here, and here and allow them to be passed in somehow.
Depending on how complicated you want to make this, pointing a variable or a config file towards the output shell script generated by pywal or wpgtk and grabbing the foreground color from that would be an awesome option. I don't think there's any option supported by the majority of terminal emulators for getting the current foreground color, unfortunately.
@mshron, I think we could start with a simple toggle between light and dark mode, like you suggested. Do you want to work on this, or should I do it?
@cantino No preference on my end. If it's not something that you wanted to make a priority, I'd be happy to work on it (probably one evening next week). But if you'd like to do it, definitely don't let me stand in your way!
I'd be a big fan of having you as an OSS contributor, so I'll let you give it a shot. π If you get stuck or can't find the time, let me know and I'll try to get it done.
π Sounds good to me
Okay, I've got the basic logic working. Now just to figure out a good color scheme π
Nice!
The ability to change theme colours will be great. I've been trying to use mcfly in the default dark terminal that's integrated into PHP Storm, and unfortunately most of the text is not visible at all, making it very difficult to use.
@DivineOmega I don't think the light mode PR will fix your problem, but it might point the way to including a more comprehensive set of color choices.
Merged in https://github.com/cantino/mcfly/commit/5be3eb8ba0122cafb7540c3750bb5ffd3aecbea6 and deployed in 0.3.0, thanks!
@mshron One issue: it looks like we lost the highlight color as you use the arrow keys. I'll try to add it back in.
Fixed in b7edc7db1197f8b32f105aa9504ce4911f5569bb
Really enjoying mcfly, thanks for making it!
With the Solarized Light color scheme, all of the un-highlighted suggestions are invisible (see attached images).
If this isn't a priority for you, I'd be happy to hack on it some time.
I poked a bit through
interface.rs
andsettings.rs
. From a cursory examination, seems like one way to add a light mode would be forsettings.rs
to look for an environment variable (MCFLY_LIGHT
?) around here, set a bool in settings, and then formenubar()
prompt()
andresults()
to act accordingly. Would that be your preferred way of propagating a setting like that?