Skip to content

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

Closed
wants to merge 2 commits into from

Conversation

armartinez
Copy link
Contributor

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

Checklist

  • I read and understood the contributing guide as well as the code of conduct
  • The issues this PR addresses are related to each other
  • My changes generate no new warnings
  • My code builds and runs on my machine
  • My changes are all related to the related issue above
  • I documented my code

Screenshots

No visual changes should occur due to this PR.

@austincondiff
Copy link
Collaborator

austincondiff commented Apr 30, 2023

@KaiTheRedNinja would your Project Navigator refactor encompass these changes?

@KaiTheRedNinja
Copy link
Contributor

When i was coding it i made sure adding/removing files worked, so yes.

@KaiTheRedNinja
Copy link
Contributor

Since this is already completed by 1243, I'm closing this PR.

@armartinez armartinez deleted the fix-new-file branch May 3, 2023 16:53
@armartinez armartinez restored the fix-new-file branch May 3, 2023 17:08
@armartinez
Copy link
Contributor Author

armartinez commented May 3, 2023

@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 ?

@austincondiff
Copy link
Collaborator

austincondiff commented May 3, 2023

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
Copy link
Contributor Author

armartinez commented May 3, 2023

@armartinez Fail enough, let's discuss @KaiTheRedNinja 🙂

@KaiTheRedNinja
Copy link
Contributor

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?

@armartinez
Copy link
Contributor Author

@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.

@armartinez armartinez deleted the fix-new-file branch June 22, 2023 11:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🐞 Can't create a New File
3 participants