Change the color of airline

[alexanderjeurissen]

This fixes #23

The default airline color used for insert mode doesn't really fit the overall look of gotham. Also the white on yellow makes it very hard to read that the active mode is insert.

This changes the color used for insert mode to magenta. See the screenshots below for (before / after)

[alexanderjeurissen]

I took another look and see your using a different color in the light line colorscheme. Don't merge this yet It's probably better to use the same colors in both colorschemes.

I'll have a look at the differences.

[alexanderjeurissen]

Also replace mode looks real weird in airline lol:

[alexanderjeurissen]

Ok so I changed several colors in the airline theme so they match that of lightline. See the screenshots below for the before / after

before:

after:

[whatyouhide]

@alexanderjeurissen hey, thanks for the contribution (thorough as always 😃). A couple of thoughts:

• I'm still not sure about the changes you did to the normal mode. It's hard to read the text now, but I'm not 100% sure it's bad because the theme is very dark; reading the filename in the middle of the modeline is still harder to read, so it may make sense. Let me think about it for a bit.
• I'm inclined to go 👎 on the change to insert mode as I like the idea of having insert mode stand out, since you shouldn't be in insert mode for long in vim and when you are, I like the idea of being reminded that you are :). Wdyt? (btw, changing the text from light to dark still is a great improvement)
• I love the new style for visual mode, let's keep it 👍
• I like the new style for replace mode, let's keep it 👍

[alexanderjeurissen]

I'm still not sure about the changes you did to the normal mode. It's hard to read the text now, but I'm not 100% sure it's bad because the theme is very dark; reading the filename in the middle of the modeline is still harder to read, so it may make sense. Let me think about it for a bit.

I've tried it for some days now and I find the text quite readable I could lighten it a bit though as I've currently used base3 instead of base4.

I'm inclined to go 👎 on the change to insert mode as I like the idea of having insert mode stand out, since you shouldn't be in insert mode for long in vim and when you are, I like the idea of being reminded that you are :). Wdyt? (btw, changing the text from light to dark still is a great improvement)

I agree with the fact that dark text is more readable on the yellow. However the point about not being in insert mode, the same can be said about visual mode because most actions where visual mode is used can be done using text-objects or using relative line numbers. That being said the other colors feel quite nice because they are also used in the colorscheme, however the yellow isn't used a lot in code highlighting and that makes it feel out of place since the yellow doesn't quite match the dark and matt theme of gotham.

One option could be to add a feature toggle that could be set in people their vimrc to make insert and visual mode stand out more. Other colorschemes have similar configuration variables where they can increase or decrease the contrast of the colorscheme for example. (see solarized and base16)

[whatyouhide]

I am 👎 against adding options for this. Let's:

• revert the "branch" and "file path" parts to the previous setting for both visual and replace, I don't like the purple-ish new setting (and I like that visual/replace/insert are the same on those parts).
• bump the text for normal mode from base3 to base4 as it's too dark for my tastes the way it is now
• revert insert to yellow background with dark foreground, as I still think it's nice that it stands out so much as you really shouldn't be in insert mode for long.

Thoughts?

[alexanderjeurissen]

Sorry for the late response.I'll address your suggestions but before I do May I ask why your against adding options for this ? options for increased contrast or diffing is quite usual in vim colorscheme land. As it allows for experimental features and/or features that only a minority of people use or want.

now to address your suggestions:

the purpelish font of the filename is because there were modifications in the file. They don't show up if you have no modifications and this color is currently being used already in those circumstances (hence is not a change I made)

I looked into it some more and I can't bump the text color for normal mode because then it's too light and unreadable. The current base 3 is'n that dark though below are some additional screenshots to show how base 4 is unreadable and the base 3 is quite nice actually ;)

I think the new insert mode stands out quite a bit because the green is very bright compared with the rest of the editor colors. I agree that you shouldn't be in insert mode a lot when writing code but I also happen to write blogs and documentation in vim using markdown and in those circumstances I'm in insert mode quite often. The yellow then kind of removes the nice dark atmosphere that makes the gotham theme such a pleasure to use.

[whatyouhide]

You can try to add an option for increased contrast (which would mean yellow insert mode) and we can see how it looks.

[alexanderjeurissen]

I'll work on that somewhere in the weekend.

[whatyouhide]

Ping @alexanderjeurissen?

[alexanderjeurissen]

@whatyouhide sorry for the delay, I'm currently using another colorscheme so recently haven't work on this. I can tackle this next weekend.

[whatyouhide]

Ping @alexanderjeurissen? :)

[alexanderjeurissen]

Sorry for the delay was really busy. I just updated the PR and will commit it shortly. I increased the visibility of the mode text by changing base3 to base2 I'm quite pleased with the result.

I added an option called: g:gotham_airline_contrast_mode which when set to 1 changes the color of insert mode to this:

[alexanderjeurissen]

I just rebased on the latest master branch.

[alexanderjeurissen]

@whatyouhide I just checked and vimscript get( always returns true if the variable is not set. This might be the correct behaviour if we want to default to the yellow insert, otherwise we need to include an explicit statement like:

let g:gotham_airline_contrast_mode = 0

in the colorscheme file, and ask people to override it. WDYT ?

[whatyouhide]

Let's default to contrasty insert, and let users override this if they want to.

[alexanderjeurissen]

@whatyouhide which of the two values do you refer to with 'contrasty insert' you mean default to yellow insert ?

if that is the case then I can leave the variable check as is and change the description in readme.

[whatyouhide]

@alexanderjeurissen yes, sorry for not being clear, I meant let's default to yellow insert and make non-yellow insert configurable. :)

[alexanderjeurissen]

@whatyouhide updated the readme, and changed the name of the config variable to: g:gotham_airline_emphasised_insert

[whatyouhide]

Thanks a lot @alexanderjeurissen! 💟