Code Monkey home page Code Monkey logo

Comments (13)

jnurmine avatar jnurmine commented on July 19, 2024

Hi, thanks for reporting this.

Looks like the g:terminal_ansi_colors is not set.

Cheers

from zenburn.

jnurmine avatar jnurmine commented on July 19, 2024

I'll soon upload a fix with nicer coloring. They have the same color order as the terminal usually has.

The terminal is not tinted towards some specific color (that could be for future work).

Looks like the terminal coloring feature needs Vim 8.2.0+
Please note, it seems one must use "set termguicolors" when running Vim in a terminal.

Please let me know if it works better, closing for now.

Cheers!

from zenburn.

 avatar commented on July 19, 2024

Hi,

Thanks, the terminal colours are as what is usually in the terminal outside Vim now'. The background is still black. I had the termguicolors set. Also, if you don't mind me saying, the relative line numbers' colouring stands out a little too much a bit distracting, I felt like the line number colours could be muted more and the background of the left gutter be a bit more grey just a shade or two different from the main background.

Thanks.

from zenburn.

jnurmine avatar jnurmine commented on July 19, 2024

Hi,

hmm, about the line numbers colouring, you are right.
Now that you pointed it out it is a standing out a bit too much.

I added a switch called g:zenburn_subdued_LineNr.

Here is a picture of it enabled in low high contrast mode in the GUI:

image

And here in high low contrast mode in the GUI:

image

Edit: typos

from zenburn.

jnurmine avatar jnurmine commented on July 19, 2024

PS. I added a fix now for the terminal colouring, use ANSI colors for Vim 8.2.0+, otherwise the older one. It looks like the Terminal highlight group disturbed the background. Now the terminal looks more as intended.

Does the terminal now look as you expected?

from zenburn.

 avatar commented on July 19, 2024

Sorry for late reply. All looks great now! Except the highlight of the numbers gutter with set cursorline. The current line's number gets underlined with yellow.

from zenburn.

jnurmine avatar jnurmine commented on July 19, 2024

Hi, can you please attach a screenshot?

I don't see the underline, with or without :set cursorline, I have tried terminal and vim -g for different versions... Do you have some other setting enabled?

from zenburn.

 avatar commented on July 19, 2024

Here is a screen shot. I launched Vim without the .vimrc, set the colorscheme and then the cursorline.

Screenshot from 2020-09-08 20-20-39

from zenburn.

jnurmine avatar jnurmine commented on July 19, 2024

Thanks, I can see it with Vim 8.2:

image

But with 8.0 no such thing:

image

The syntax file is exactly the same in both cases.

It looks like Vim overrides cterm=underline always, possibly due to commit 017ba07fa2cdc5 in 8.1.2029.

Not sure yet how to solve this, explicitly specifying cterm doesn't help.

from zenburn.

jnurmine avatar jnurmine commented on July 19, 2024

Looks like explicit cterm=none works, but the colorscheme must be loaded at Vim startup time i.e. Vim must be restarted for this change to take effeect.

There's a fix now, can you please check if it's OK?

from zenburn.

 avatar commented on July 19, 2024

Your fix works, thanks. Following your code above, I've put this right at the end of the .vimrc

set cursorline    
highlight CursorLineNR cterm=NONE ctermfg=234 ctermbg=234 guifg=#424242 guibg=#1b1b1b

I am trying to make the line highlight not effect the gutter at all, but I can't get the background colour right, I got the colours from your zenburn.vim file. What is the correct gui background colour for the gutter? Do you know if there is an easier way to disable the highlight in the gutter altogether?

from zenburn.

jnurmine avatar jnurmine commented on July 19, 2024

Hi, there should not be need to specify highlight CursorLineNR since Zenburn has the cterm=none now.

Anyway, you should get a newer Vim, like 8.2 and then do :set cursorlineopt=line to avoid highlighting the gutter.

Edit: just to be clear, grab the newest Zenburn and you get commit 4ec3795 "Fix underlined CursorLineNr in gutter" which does the cterm=none.

from zenburn.

 avatar commented on July 19, 2024

Ah, that's great! Thank you so much!

from zenburn.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.