Skip to content

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 a team, artonge, juliusknorr, marcoambrosini 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

Signed-off-by: Simounet <contact@simounet.net>
@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