-
-
Notifications
You must be signed in to change notification settings - Fork 4.7k
fix(files): right click actions menu flicker #51863
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
8.24 will release today so should work with it as well |
but I don't wanna update all stablexx branches with latest nc/vue, will take forever to be merged as always |
|
/compile rebase |
|
/backport! to stable31 |
|
/backport! to stable30 |
|
/backport! to stable29 |
Signed-off-by: skjnldsv <[email protected]>
Signed-off-by: nextcloud-command <[email protected]>
2b330cb to
2f58abb
Compare
Fix #51624
Best fix is with nextcloud-libraries/nextcloud-vue#6683, but for backports and the fact that we want this in today, let's use a
setTimeoutuntil deployed 🚀Context
Explanations
The apply-hide event was wrongly used (because we have our own additional method called after-hide
The timing was also wrong (the closed event was triggered BEFORE the popup was actually hidden in the dom.
Solution