CodeEdit icon indicating copy to clipboard operation
CodeEdit copied to clipboard

Improved File Handling System

Open Wouter01 opened this issue 2 years ago • 1 comments

Description

Rewrite of the file handling system.

The current file handling system starts to get slow for larger projects. This is noticeable to the user by the slow startup times of codeedit, slower file moving in larger projects, etc. A new file handling system is added which is more performant. It also makes more use of the capabilities of NSDocument, simplyfying things.

Related Issues

  • #1336

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

Wouter01 avatar Jun 20 '23 02:06 Wouter01

Wowzer

Rachmaye avatar Jun 20 '23 05:06 Rachmaye

What is the status of this PR?

austincondiff avatar Aug 16 '23 18:08 austincondiff

What is the status of this PR?

It's 80% done, but I currently don't have the time to resume working on it

Wouter01 avatar Aug 16 '23 18:08 Wouter01

So close! That is okay, I totally understand. Do you have any idea when you might get the time in the future? Depending, it may be best to close this for the time being and reopen when that time comes. What do you think?

austincondiff avatar Aug 16 '23 18:08 austincondiff

So close! That is okay, I totally understand. Do you have any idea when you might get the time in the future? Depending, it may be best to close this for the time being and reopen when that time comes. What do you think?

Maybe in September, but I can't promise anything 😅. It's fine for me to close this PR for now, but it's probably best that a linked issue is created so the work doesn't get lost. Also, branches are removed from time to time, so I'm hoping this one doesn't get deleted. I have a local copy though

Wouter01 avatar Aug 16 '23 18:08 Wouter01

In that case, we can keep this PR open. It may help to keep it updated from time to time though so you don't end up with major conflicts down the road.

austincondiff avatar Aug 16 '23 18:08 austincondiff

Closing this as discussed with @Wouter01. This will be implemented but the current fork is too far behind CE to be merged without a lot of pain.

thecoolwinter avatar Sep 23 '23 00:09 thecoolwinter