-
Notifications
You must be signed in to change notification settings - Fork 27
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
Reactor/files app #556
Reactor/files app #556
Conversation
Signed-off-by: Nikolas Rimikis <rimikis.nikolas@gmail.com>
Why is it in draft if I should already review? 👀 |
Because I love the UX of github 🙄 |
Ah yeah it remembers what type of PR you last submitted. I also hate it so much lol |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As far as I understand the performance improvements mostly come from caching widgets or am I wrong here?
Signed-off-by: Nikolas Rimikis <rimikis.nikolas@gmail.com>
Signed-off-by: Nikolas Rimikis <rimikis.nikolas@gmail.com>
Signed-off-by: Nikolas Rimikis <rimikis.nikolas@gmail.com>
9dc4e42
to
94577d2
Compare
94577d2
to
95084d1
Compare
… sealed Task class Signed-off-by: Nikolas Rimikis <rimikis.nikolas@gmail.com>
95084d1
to
a7b0dda
Compare
improves the files app performance as mentioned in #445.
#282 will still gain us a bit more.
@provokateurin please review thoroughly :)