-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Fixed error adding new file or folder from the outline menu #1236
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
Conversation
@KaiTheRedNinja would your Project Navigator refactor encompass these changes? |
When i was coding it i made sure adding/removing files worked, so yes. |
Since this is already completed by 1243, I'm closing this PR. |
@KaiTheRedNinja I tested you changes and it still doesn't work properly since it keeps the current implementation that relies on the OS filewatcher. Can you maybe confirm @austincondiff ? |
I will revert to what @KaiTheRedNinja has to say as he is probably a little more familiar than I am. That said, if your changes use a more reliable or performant convention then we should probably at least look into why you are recommending why we use it over what was previously put in place. |
@armartinez Fail enough, let's discuss @KaiTheRedNinja 🙂 |
the file watcher is so that changes made within finder (outside of ce) cause updates as well. Is there some issue with that system that i’m not aware of? Also, are you on the discord? |
@KaiTheRedNinja I am in the discord, we can discuss there, the issue is that it is very slow, at least in my system. I think it should be instant like any other editor. |
Description
This PR fixes the error where adding a new folder or file from the outline menu didn't reflect on the outline itself.
Related Issues
New File
#1146Checklist
Screenshots
No visual changes should occur due to this PR.