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

Error after reload vimrc #132

Closed
bennyyip opened this issue Apr 22, 2018 · 2 comments
Closed

Error after reload vimrc #132

bennyyip opened this issue Apr 22, 2018 · 2 comments

Comments

@bennyyip
Copy link
Contributor

bennyyip commented Apr 22, 2018

I copy this line from doc to my vimrc:

let g:Lf_StlSeparator = { 'left': "\ue0b1", 'right': "\ue0b2"}

The font field is missing, which should be

let g:Lf_StlSeparator = { 'left': "\ue0b1", 'right': "\ue0b2", 'font': ''}

LeaderF works well until :source ~/.vim/vimrc. When I invoke a LeaderF command, :messages gives following output:

Messages maintainer: Bram Moolenaar <[email protected]>
"~/.vim/vimrc" 945L, 29485C
Error detected while processing function leaderf#Mru#startExpl[2]..leaderf#LfPy:
line    1:
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/home/benyip/.vim/bundle/LeaderF/autoload/leaderf/python/leaderf/manager.py", line 731, in startExplorer
    self._getInstance().enterBuffer(win_pos)
  File "/home/benyip/.vim/bundle/LeaderF/autoload/leaderf/python/leaderf/manager.py", line 225, in _getInstance
    self._afterExit)
  File "/home/benyip/.vim/bundle/LeaderF/autoload/leaderf/python/leaderf/instance.py", line 40, in __init__
    self._highlightStl()
  File "/home/benyip/.vim/bundle/LeaderF/autoload/leaderf/python/leaderf/instance.py", line 64, in _highlightStl
    lfCmd("call leaderf#colorscheme#highlight('{}')".format(self._category))
vim.error: Vim(if):E716: Key not present in Dictionary: font
Error detected while processing function leaderf#Mru#startExpl:
line    2:
E171: Missing :endif
1 more line

E474: Invalid argument

This seems like a regression, it didn't happen before. Should we update the doc?

@Yggdroot
Copy link
Owner

Thanks for reporting, this issue seems the same as issue #129, I 'll try to fix it.

Yggdroot added a commit that referenced this issue Apr 23, 2018
@Yggdroot
Copy link
Owner

Yggdroot commented Apr 23, 2018

Fixed. Now you don't have to add 'font': '' any more.

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