-
Notifications
You must be signed in to change notification settings - Fork 630
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
Proposal: Force all themes to use lowercase names #66
Comments
do you have any data on how many existing color schemes violate this? |
Piggybacking on this issue, I was trying out monokai today. To my surprise it didn't load:
so I changed to
after that I did using:
so yeah, this seems to work well on case sensitive filesystems </sarcasm> |
My current opinion is that this repo is true to upstream. If there is an issue preventing a colorscheme from being used maybe we take out the offending theme until upstream is fixed. I'm open to discussion, but it might take a while. |
This is causing many issues with macos since the HFS+ file system is case-insensitive:
|
And there is the same issue with Git always report dirty state of repo, and there is no way to checkout clean master branch. Has anybody forked this repository to fix this ↑ issue already? |
I am not sure why is this not merged? The fix is obvious and simple, and it is impacting all case-insensitive filesystem (e.g. default APFS, Windows) |
I second the vote for this! |
sorry all for the late reply, I was able to reproduce much of the complaint. I just did a fresh checkout on mojave and did So it seems there is an issue with git+apfs; an issue with vim's autocompletion. Is there any upstream knowledge around these? Mandating all files are lower case will solve these specific issues, but will the root causes manifest outside of latin case? |
I agree an upstream fix would be better but I wonder how long that would take to trickle out. |
Someone want to get a branch (or three) up (apologies if one exists and I dismissed it) with changes specific to these files. Let's get the changes out upstream too. Considering upstream traction we can work towards getting these into master. Thanks for all the concern and patience folks. |
I've noticed that the themes using an uppercased name are missing out on tab completion (unless uppercase letter is specified) or even just basic colorscheme name cycling. This would be an easy change but would force existing users to update their vimrc.
The text was updated successfully, but these errors were encountered: