Skip to content

fix(admin-mods): modernize mods list and forms (#1552) - #1552

Open
maxijabase wants to merge 2 commits into
sbpp:mainfrom
maxijabase:feat/mods-list-compact-actions
Open

fix(admin-mods): modernize mods list and forms (#1552)#1552
maxijabase wants to merge 2 commits into
sbpp:mainfrom
maxijabase:feat/mods-list-compact-actions

Conversation

@maxijabase

@maxijabase maxijabase commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Description

Brings the admin Mods surfaces in line with the rest of the v2 panel list / form chrome.

List mods (?p=admin&c=mods&section=list)

  • Dense table--compact desktop table (less vertical padding per row).
  • Icon-only row actions (pencil / trash-2) with data-tooltip + aria-label, matching banlist / admins.
  • Page-level Mods (N) title outside the card (same shape as Admins).
  • Mobile card mirror (.mods-list-cards) so the table is not hidden and does not force horizontal scroll under 768px. Delete removes both the desktop row and the mobile card.

Add mod (section=add)

  • Page-level Add mod title + subtitle outside the card (no title trapped in card__header).

Edit mod (o=edit)

  • Removes the empty AdminTabs Back strip that left dead space above the form (redundant with the footer Back next to Save).
  • Page-level Edit mod · <name> title + subtitle, matching Edit admin / Edit ban.

Motivation and Context

Mods list / add / edit still used the older “title inside the card + text Edit/Delete + keep-mobile table” shape after bans, comms, and admins moved to compact tables, icon actions, page titles, and mobile cards. This PR closes that visual and responsive gap without changing mod semantics or APIs.

How Has This Been Tested?

  • Local Docker panel at :8080
  • Manual check: list (desktop + narrow viewport), add, edit
  • Existing E2E/integration hooks (mod-row, deletemod-btn, mod-delete dialog) kept; count badge format now (N) like admins

Screenshots (if appropriate):

Before

RB3SxxykDV.mp4

After

chrome_XpOmzWRTum.mp4

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)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.

@maxijabase
maxijabase requested a review from a team August 5, 2026 23:33
@Rushaway Rushaway changed the title Polish admin Mods list, add, and edit chrome feat(admin-mods): modernize mods list and forms (#1552) Aug 9, 2026
@Rushaway Rushaway changed the title feat(admin-mods): modernize mods list and forms (#1552) fix(admin-mods): modernize mods list and forms (#1552) Aug 9, 2026
@Rushaway
Rushaway enabled auto-merge August 9, 2026 16:32
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.

2 participants