Skip to content
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

Fix trashbin files view sticky action bar #28754

Merged

Conversation

Simounet
Copy link
Member

@Simounet Simounet commented Sep 7, 2021

Hi there,

It's a simple quick fix for the action bar of the trashbin view. When you selected some files, the sticky position wasn't activated on scroll and you had to go to the top to have access to the action menu.

Have a nice day.

Copy link
Contributor

@szaimen szaimen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix!
Seems to work :)

single command to try
docker run \
-e SERVER_BRANCH=Simounet:fix/files-trashbin-action-sticky-bar \
-p 127.0.0.1:8443:443 \
--name nextcloud-easy-test \
ghcr.io/szaimen/nextcloud-easy-test:latest

For further infos see https://github.com/szaimen/nextcloud-easy-test#nextcloud-easy-test-instance

@szaimen szaimen added 3. to review Waiting for reviews design Design, UI, UX, etc. enhancement feature: trashbin labels Sep 9, 2021
@szaimen szaimen added this to the Nextcloud 23 milestone Sep 9, 2021
@szaimen szaimen requested review from marcoambrosini, a team, juliusknorr, artonge and skjnldsv and removed request for a team September 9, 2021 09:25
@artonge
Copy link
Contributor

artonge commented Sep 9, 2021

DCO, then good to merge :)

@szaimen
Copy link
Contributor

szaimen commented Sep 9, 2021

/backport to stable22

@szaimen
Copy link
Contributor

szaimen commented Sep 9, 2021

/backport to stable21

@szaimen
Copy link
Contributor

szaimen commented Sep 9, 2021

/backport to stable20

@Simounet Simounet force-pushed the fix/files-trashbin-action-sticky-bar branch from 9e0b01a to ce570d0 Compare September 9, 2021 09:57
@szaimen szaimen added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Sep 9, 2021
@artonge artonge merged commit 9be9393 into nextcloud:master Sep 9, 2021
@Simounet Simounet deleted the fix/files-trashbin-action-sticky-bar branch September 9, 2021 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish design Design, UI, UX, etc. enhancement feature: trashbin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants