Skip to content

Add Page Size Selector to Requests view#454

Merged
cycomachead merged 2 commits into
mainfrom
cycomachead/150-on-the-pending-requests-view-please-set-the-default-per-page-to-50-add-a-page-size-selector-25-50/1
Jun 26, 2026
Merged

Add Page Size Selector to Requests view#454
cycomachead merged 2 commits into
mainfrom
cycomachead/150-on-the-pending-requests-view-please-set-the-default-per-page-to-50-add-a-page-size-selector-25-50/1

Conversation

@cycomachead

Copy link
Copy Markdown
Contributor

General Info

Changes

Updates the pending requests (instructor index) table to improve usability for larger datasets:

  • Sets the default page length to 50 rows (previously defaulted to DataTables' built-in 10)
  • Adds a page size selector with options: 25, 50, 100, 200, All
  • Re-adds the pageLength feature to the topStart layout cell so the selector renders alongside the existing Columns/Batch Edit buttons

All changes are client-side only in requests_controller.js — no Ruby or database changes needed.

Note for reviewers: The page-size dropdown and the Columns/Batch Edit buttons now share the top-left cell. Worth a quick visual check that the layout looks clean, though DataTables-bs5 handles this automatically.

Testing

Manually verify on the pending requests view that:

  1. The table defaults to 50 rows per page on load
  2. The page-size dropdown appears and offers 25, 50, 100, 200, and All options
  3. Selecting a different page size updates the table correctly
  4. The existing Columns and Batch Edit buttons still render and function correctly alongside the new dropdown

Documentation

No documentation changes required.

Checklist

  • Name of branch corresponds to story

Superconductor Ticket Implementation | App Preview | Guided Review

cycomachead and others added 2 commits June 24, 2026 06:49
- Set default page length to 50
- Add page size selector with options for 25, 50, 100, 200, and All
- Enable pageLength control in the DataTables layout configuration

Co-authored-by: Claude Code <noreply@anthropic.com>
@cycomachead cycomachead changed the title On the 'pending requests' view, please * set the default per page to 50 * Add a page size selector: 25, 50, 100, 200, All Add Page Size Selector to Requests view Jun 26, 2026
@cycomachead cycomachead merged commit 42d5e3f into main Jun 26, 2026
11 checks passed
@cycomachead cycomachead deleted the cycomachead/150-on-the-pending-requests-view-please-set-the-default-per-page-to-50-add-a-page-size-selector-25-50/1 branch June 26, 2026 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant