-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Renaming a file opens the sidebar #18613
Comments
This comment has been minimized.
This comment has been minimized.
Took a second to look at this again. Looks like its slightly more complicated. // show defaults to true
show = _.isUndefined(show) || !!show
if (!show && OCA.Files.Sidebar.file === '') {
return
} server/apps/files/js/filelist.js Lines 633 to 640 in 9e7f52d
|
I guess not @gary-kim ! |
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Steps to reproduce
Expected behaviour
File is renamed and that’s that.
Actual behaviour
File is renamed but also the sidebar is opened for some reason. That’s unnecessary and especially annoying when on a small screen and you want to rename some more files and you have to close the sidebar again and again.
Server configuration
Latest server beta
The text was updated successfully, but these errors were encountered: