-
Notifications
You must be signed in to change notification settings - Fork 108
fix(workspace): toggle right away and only render if needed #7501
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
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #7501 +/- ##
=======================================
Coverage 59.64% 59.65%
=======================================
Files 495 495
Lines 37869 37866 -3
Branches 1090 1090
=======================================
+ Hits 22588 22589 +1
+ Misses 15174 15170 -4
Partials 107 107 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
196db02 to
188f706
Compare
Awaiting the request for the file lead to race conditions: * Enter directory with rich workspace, loading is triggered. * Navigate to directory without rich workspace. * Loading finishes - rich workspace was shown. Also clean up some minor things: * `this.creating` was not used at all. * The Editor does not emit `error`. Signed-off-by: Max <max@nextcloud.com>
9a98dd1 to
ce387af
Compare
The header is just hidden if enabled is false. But for the workspace this leads to continuous requests updating the content. Signed-off-by: Max <max@nextcloud.com>
ce387af to
1a8f287
Compare
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.
Nice finding! I didn't test, but I guess you did. Thanks for being so persistent here!
|
Probably also a fix worth backporting, no? |
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.
Make sense!
|
/backport to stable31 |
fixes #7505