Skip to content

Mobile view: hide checkbox and status columns on requests page#365

Open
ba88im wants to merge 11 commits intoberkeley-cdss:mainfrom
cs169:ba88im/mobile-view-requests-page
Open

Mobile view: hide checkbox and status columns on requests page#365
ba88im wants to merge 11 commits intoberkeley-cdss:mainfrom
cs169:ba88im/mobile-view-requests-page

Conversation

@ba88im
Copy link
Copy Markdown
Contributor

@ba88im ba88im commented Apr 2, 2026

Summary

  • Adjusts data-priority values on the instructor requests table so that on small screens, only the Actions, Assignment, and Name columns are shown (checkbox and status columns are hidden via DataTables responsive).
  • Hides bulk approve/reject buttons on mobile using Bootstrap d-none d-md-block.

Test plan

  • Open the instructor requests page on a mobile device or narrow browser window
  • Verify only Actions, Assignment, and Name columns are visible
  • Verify checkbox column and Status column are hidden
  • Verify bulk action buttons are hidden on mobile
  • Verify full table still renders correctly on desktop

🤖 Generated with Claude Code

@superconductor-for-github
Copy link
Copy Markdown

superconductor-for-github bot commented Apr 2, 2026

@ba88imSuperconductor finishedView implementation | App Preview


Standing by for instructions.

noahnizamian and others added 4 commits April 1, 2026 17:27
Adjust DataTables data-priority values so only Actions, Assignment,
and Name columns remain visible on small screens. Bulk action buttons
are also hidden on mobile via Bootstrap responsive utility classes.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@ba88im ba88im force-pushed the ba88im/mobile-view-requests-page branch from f3d413c to e8e003f Compare April 2, 2026 00:27
@cycomachead
Copy link
Copy Markdown
Contributor

Thanks! It looks like there's a couple of PRs worth of work in here? Can you please sync your branch and address the merge conflict.

@cycomachead
Copy link
Copy Markdown
Contributor

@ba88im Can you please pull in the upstream changes and resolve the merge conflicts?

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.

4 participants