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

Current argument in function signature preview of jedi-vim not properly highlighted #14

Closed
mkalinski opened this issue Jan 20, 2017 · 5 comments

Comments

@mkalinski
Copy link

Jedi-vim is a popular Vim plugin for python programming.

One of its features is a live preview of the signature of the function that is being currently typed. Current position in the argument list is highlighted, but in nord-vim this is not visible. That's because jedi-vim assumes that TabLine and CursorLine groups are highlighted differently, but they aren't in nord-vim.

From https://github.com/davidhalter/jedi-vim/blob/master/after/syntax/python.vim:

    if exists('g:colors_name')
        hi def link jediFunction CursorLine
        hi def link jediFat TabLine
    else
        hi jediFunction term=NONE cterm=NONE ctermfg=6 guifg=Black gui=NONE ctermbg=0 guibg=Grey
        hi jediFat term=bold,underline cterm=bold,underline gui=bold,underline ctermbg=0 guibg=#555555
    endif

Either TabLine and CursorLine should be highlighted differently, or the plugin-specific jediFunction and jediFat overridden (jediFat is the one that should stand out more).

@arcticicestudio
Copy link
Contributor

arcticicestudio commented Jan 24, 2017

I'll test it next weekend, but as far as I've seen your suggestion with overriding the methods sounds like a clean solution.
Changing the TabLine or CursorLine would break many other plugins that rely on this groups.

@arcticicestudio
Copy link
Contributor

I've tested the auto completion feature today and for me the list looks fine.
The currently selected entry has a lighter background and the font using the main accent theme color nord8.

ghi-14-scrot-auto-completion

Can you please post a screenshot of your view, your .vimrc config and maybe some example code so I can try to reproduce it.

And by the way, this plugin is awesome! 😄
I'll definitely use it for my Python projects, thanks 👍

@mkalinski
Copy link
Author

Yeah, this display is alright, I meant something different. I'll provide the screenshots.

Here is the function call signature prompt in the default theme:
jedi-default

Meanwhile in nord:
jedi-nord

@arcticicestudio
Copy link
Contributor

arcticicestudio commented Jan 30, 2017

Thanks, this looks definitely worng 😆
I'll add support for it as soon as I've got some freetime for it 😄

arcticicestudio added a commit that referenced this issue Feb 4, 2017
This actually includes two groups which had to be adjusted.
Both are used for a parameter popup in functions. The background color
and the color of the current parameter at the cursor are now colored
explicitly instead of the default syntax link to the "TabLine" and
"CursorLine" groups.
@arcticicestudio
Copy link
Contributor

Implemented optimized styles for the jediFunction and jediFat groups.

Before

After

@arcticicestudio arcticicestudio added this to the 0.4.0 milestone Jul 10, 2017
@arcticicestudio arcticicestudio removed their assignment Sep 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants