Skip to content

Implemented virtual scroll in trash bin#2809

Merged
PVince81 merged 1 commit intomasterfrom
feature/trashbin-virtual-scroll
Jan 8, 2020
Merged

Implemented virtual scroll in trash bin#2809
PVince81 merged 1 commit intomasterfrom
feature/trashbin-virtual-scroll

Conversation

@LukasHirt
Copy link
Copy Markdown
Collaborator

Description

Re-use FileList component with the virtual scroll inside of trash bin.

Related Issue

How Has This Been Tested?

  • test environment: Drone

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:

@LukasHirt LukasHirt added the Status:Needs-Review Needs review from a maintainer label Jan 3, 2020
@LukasHirt LukasHirt requested a review from PVince81 January 3, 2020 09:12
@LukasHirt LukasHirt self-assigned this Jan 3, 2020
@LukasHirt LukasHirt force-pushed the feature/trashbin-virtual-scroll branch 2 times, most recently from bbbf291 to 2838d93 Compare January 3, 2020 10:19
Adjusted selectors

Fixed toggle selector

Fixed selectors
@LukasHirt LukasHirt force-pushed the feature/trashbin-virtual-scroll branch from 2838d93 to f3a4084 Compare January 7, 2020 15:22
Copy link
Copy Markdown
Contributor

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

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

Code looks fine 👍

Glad to see that Trashbin is now also using the FileList base view

@PVince81 PVince81 merged commit f25b04a into master Jan 8, 2020
@delete-merged-branch delete-merged-branch bot deleted the feature/trashbin-virtual-scroll branch January 8, 2020 08:29
@fschade fschade mentioned this pull request Apr 29, 2022
25 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Status:Needs-Review Needs review from a maintainer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement virtual scroll in trashbin

2 participants