Use filetreelist crate
No longer safe Status inside of FileTreeItem.
For Status storage, now introduced a new Item, which contains Status alongside FileTreeItem.
Note that FileTreeItemKind is no longer publicy available - thus we cannot store this enum anywhere anymore.
I followed the checklist:
- [ ] I added unittests
- [x] I ran
make checkwithout errors - [x] I tested the overall application
- [x] I added an appropriate item to the changelog
~~So far change component shows no diff - not sure why~~
I think for being able to write unittests I would need to extract a lot of traits for components.
All good the tree component itself is pretty well tested
While reviewing noticed that it still has its own FileTreeItems instead of using the one from filetreelist-crate. We should make it use the crate one
putting in draft till the next round of reviews
While reviewing noticed that it still has its own
FileTreeItemsinstead of using the one fromfiletreelist-crate. We should make it use the crate one
The reason is that filetreelist crate knows nothing about status. If you want to use it I see 2 basic ways of doing it:
- Introduce status to
filetreelistcrate - Have a separate "list" for status
How do you want handle that?
This issue has been automatically marked as stale because it has not had any activity half a year. It will be closed in 14 days if no further activity occurs. Thank you for your contributions.