Skip to content

feat(frontend): refine ban list and communications block list - #1546

Merged
Rushaway merged 8 commits into
sbpp:mainfrom
maxijabase:improve/banlist-comms-ux
Aug 4, 2026
Merged

Rushaway merged 8 commits into
sbpp:mainfrom
maxijabase:improve/banlist-comms-ux

Conversation

@maxijabase

@maxijabase maxijabase commented Aug 4, 2026 •

Copy link
Copy Markdown
Contributor

Description

UX polish pass across the public Ban List and Communications Block List:

  • Denser tables with icon-only row actions (tooltips + aria-label), compact avatars, and tighter chrome
  • Header actions aligned between banlist and comms (btn--sm, Lucide icons; banlist search icon parity)
  • Advanced Steam search always uses partial LIKE match (drops Exact/Partial toggles; advType=steamid kept as legacy alias)
  • Delete confirms use styled <dialog>s instead of window.confirm (ban delete, comm delete, comment delete)
  • Comment chips sit on the same line as the player name (icon + count); name truncates so the chip does not wrap alone
  • Avatar stays top-aligned next to the name when a comment disclosure is open

Motivation and Context

Banlist / commslist rows felt tall and uneven: comment badges forced an extra line under the player name, delete flows still used native confirms, and header / row action chrome drifted between the two lists. This brings both surfaces onto the same denser, more predictable pattern.

How Has This Been Tested?

  • Manual checks on banlist + commslist (desktop): density, header buttons, inline comment chips, expanded-comment avatar alignment, delete dialogs
  • PHPUnit assertions added/updated in BanlistCommentsVisibilityTest (head row + no __label)
  • Related E2E touch-ups in comms-affordances / comment-delete-dispatcher

Screenshots

Before

image
image

After

image
image

Comment chip (collapsed / expanded)

image
image

Delete dialogs

image

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.

@Rushaway Rushaway changed the title Improve banlist and comms list density and chrome feat(frontend): refine ban list and communications block list Aug 4, 2026
@Rushaway
Rushaway added this pull request to the merge queue Aug 4, 2026
Merged via the queue into sbpp:main with commit 6c2d7cf Aug 4, 2026
7 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 4, 2026
@maxijabase
maxijabase deleted the improve/banlist-comms-ux branch August 31, 2026 17:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants