Comments (8)
@dangibson
Hi,
May I ask why you closed your PR?
I've recently submitted a PR to solve another problem with case-insensitive file systems when using the move operation(#1375), and introduced a flag called g:NERDTreeCaseInsensitiveFS
instead of guessing based on the operating system as both macOS and Windows can support case sensitive file systems.
I think with the use of this flag we can also fix this problem across platforms instead of specifically on Windows machines.
Would you like to reopen your PR and use this new flag to solve this issue? Even if you don't want to work on it anymore I will create a PR based on your commits to keep the credits for them.
Let me know if you want to continue working on it.
from nerdtree.
I fixed the issue I had and it's been working fine for me. I didn't think I closed it.
from nerdtree.
Oh, I don't know why I thought it was closed, what do you think about using the same flag in your PR? do you think it's a good idea?
from nerdtree.
I would probably default to assuming it is case insensitive instead of sensitive. While that is technically wrong for linux, it's unlikely people would have a file or folder called "abc" and another called "Abc" in the same folder. Although the end result is the same, the difference is that on Windows you wouldn't have to set the flag manually, and on linux you probably wouldn't need to change it so it would be a slightly better out of the box experience.
If you have a pull request that resolves the issue then I'm happy to scrap mine - no credit needed. I resolved the issue for me and submitted a PR but nothing happened with it and I don't really have an interest in working further on it.
from nerdtree.
I've set it to Case sensitive by default to make it backward compatible, this way we don't break anything for users who are used to this behavior and may have put their workspace(although stupid) around this thing.
You can have something like this in your .vimrc file if you work with the same vim configuration on different machines
if nerdtree#runningWindows()
let g:NERDTreeCaseInsensitiveFS = 1
elseif has('gui_mac') || has('gui_macvim') || has('mac') || has('osx')
let g:NERDTreeCaseInsensitiveFS = 1
else
let g:NERDTreeCaseInsensitiveFS = 0
endif
If we assume case sensitivity wrong in critical places it can cause files to be overwritten which is not a good thing to happen,
Not showing some files or throwing an error is much safer than overwriting it that's why I thought it is much better to give the choice to the end user. We can reuse this flag for many purposes, And in the future, we have the option to turn it on by default for Windows and macOS as they have case-insensitive file systems by default.
Let me know what you think about it.
from nerdtree.
I think on Windows / mac it should default to insensitive without the user needing to explicitly set it since that is what the users would reasonably expect to happen. If it defaults to sensitive then they would consider it a bug and spend time searching for the solution.
I think it is better to set the flag now instead of 'in the future' since 'in the future' there would be resistance to changing it since it wouldn't be backwards compatible.
from nerdtree.
I don't have a strong take on whether or not this flag should be on by default or not, But I think you agree with the fact that this flag is needed in the first place, Right?
from nerdtree.
The flag is a good idea - testing a flag is easier than testing is windows or is osx or is mac or is macvim etc
from nerdtree.
Related Issues (20)
- Can't find how to actually select the dropped-down result
- `(m)ove the current node` shortcut doesn't treat filepaths as case sensitive HOT 10
- Unable to open file on newly opened tree after the default tree is closed HOT 4
- Adding change logs for few last merged PRs and create tags for them HOT 10
- How to set the padding width between arrow and the icon HOT 4
- Issue #1368 remote authentication required…. HOT 2
- What's the remap api for nerdtree menu? HOT 2
- Change license to something more standard HOT 3
- Customize key to open quick help menu HOT 3
- Error when loading a vim session: BufLeave NERD_tree_* autocommand: Variable b:NERDTree does not exist HOT 3
- E1208: -complete used without allowing arguments HOT 2
- BUG: NERDTree Hijacked Movement Keys HOT 6
- Installing NERDTree with Vim 9 on Windows 10 HOT 4
- Error detected while processing WinEnter Autocommands for "*"..function <SNR>38_HandleOnlyWindow[13]..<SNR>38_CloseWindow: HOT 1
- Add user choice to follow symlinks or keep them HOT 4
- Installation??? HOT 4
- highlight the current filename in the tree window after following a tag HOT 1
- NERDTree.TreeNotOpen thrown when opening Neovim into NERDTree HOT 4
- Error detected while processing BufEnter Autocommands for "*" with the "Close the tab if NERDTree is the only window remaining in it." config HOT 9
- Error detected while processing function nerdtree#ui_glue#invokeKeyMap[1]..96[18]..95[3]..<SNR>185_openHSplit[1]..125[1]..141[3]..197[6]..198[17]..33: HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
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.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from nerdtree.