-
Notifications
You must be signed in to change notification settings - Fork 990
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
CtrlP theme #1489
Comments
You need to have at least modifications to three files:
Most likely also powerline/segments/vim/plugin/ctrlp.py with contents similar (functionally) to commandt.py in the same directory. |
Ah, and also tests/test_ctrlp.vim (see tests/test_commandt.vim). |
Thanks, I were able to implement it and it's functioning alright, now how can I migrate it from "powerline/matchers/vim/plugin/ctrlp.py" to my home directory? I also needed to write a "powerline/segments/vim/plugin/ctrlp.py" for utility functions. but I want these 2 files in my home directory instead of my python's site-packages folder. |
I thought you were going to create a PR. Powerline does not put any restrictions on which modules exactly need to have the matchers and segments, so you could put it in any module, not necessary These are not configuration files, so you should not put them into |
thanks for info, there's only one thing blocking me to create the pull request. my ctrlp segment module relies on setting ctrlp status function to my custom function, the ctrlp variable that should be set for this purpose is g:ctrlp_status_func, is it alright to set this variable with calling vim.command in my initialize function in ctrlp module or maybe it's an antipattern regarding powerline plugin's guidelines. |
@sassanh Cannot say without actually seeing the code. Should be OK assuming that you do not set this variable until ctrlp buffer actually appears. |
@ZyX-I I just created the PR. |
Is it possible to have a ctrlp theme for powerline? It'd be great if you could lead me to write one if no one with the knowledge has the time to write it himself.
Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.
The text was updated successfully, but these errors were encountered: