Skip to content

Fixed error adding new file or folder from the outline menu#1236

Closed
armartinez wants to merge 2 commits intoCodeEditApp:mainfrom
armartinez:fix-new-file
Closed

Fixed error adding new file or folder from the outline menu#1236
armartinez wants to merge 2 commits intoCodeEditApp:mainfrom
armartinez:fix-new-file

Conversation

@armartinez
Copy link
Copy Markdown
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
Copy Markdown
Collaborator

austincondiff commented Apr 30, 2023

@KaiTheRedNinja would your Project Navigator refactor encompass these changes?

@KaiTheRedNinja
Copy link
Copy Markdown
Contributor

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

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

armartinez commented May 3, 2023

@armartinez Fail enough, let's discuss @KaiTheRedNinja πŸ™‚

@KaiTheRedNinja
Copy link
Copy Markdown
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
Copy Markdown
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