Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

NEW Add paginator component #1909

Merged

Conversation

emteknetnz
Copy link
Member

@emteknetnz emteknetnz commented Feb 27, 2025

Issue #1892

This component replaces the Griddle paginator that was used for paginating results in the following places:

  • ThumbnailView.js
  • TableView.js
  • HistoryViewer.js

@emteknetnz emteknetnz mentioned this pull request Feb 27, 2025
3 tasks
@emteknetnz emteknetnz force-pushed the pulls/3.0/paginator branch 3 times, most recently from ed1880f to aae7113 Compare March 3, 2025 22:38
@emteknetnz emteknetnz marked this pull request as ready for review March 3, 2025 23:19
Copy link
Member

@GuySartorelli GuySartorelli left a comment

Choose a reason for hiding this comment

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

Not clear whether silverstripe/silverstripe-versioned-admin#391 (review) will require changes here so I'll hold off approving

@GuySartorelli
Copy link
Member

Where is ThumbnailView.js used so I can test it?

@@ -0,0 +1,86 @@
.paginator-footer > div {
Copy link
Member Author

Choose a reason for hiding this comment

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

@emteknetnz emteknetnz force-pushed the pulls/3.0/paginator branch from aae7113 to eb70137 Compare March 4, 2025 21:06
@emteknetnz
Copy link
Member Author

Where is ThumbnailView.js used so I can test it?

It's the default view in asset-admin i.e. large thumbnails, as opposed to the TableView which you toggle to

Copy link
Member

@GuySartorelli GuySartorelli left a comment

Choose a reason for hiding this comment

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

LGTM

@GuySartorelli
Copy link
Member

Oop looks like new conflicts 😅

@emteknetnz emteknetnz force-pushed the pulls/3.0/paginator branch from 765996d to 7e7c561 Compare March 5, 2025 03:41
@GuySartorelli GuySartorelli merged commit 223c1b5 into silverstripe:3.0 Mar 5, 2025
15 checks passed
@GuySartorelli GuySartorelli deleted the pulls/3.0/paginator branch March 5, 2025 04:45
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants