review · segments
Fix a11y/badges/buttons in admin string-hash and texts views
claude 89 events 1 segments daisyui-removal-audit
segment 1 of 1
Apply a11y, badge, and button fixes to admin string-hash and texts views
The assistant read all target files, then systematically applied the worklist transformations: converted clickable divs to buttons with proper disabled handling, replaced hand-rolled status badges with <x-status-pill> component, cleaned up flux button overrides, removed empty placeholder columns, and verified Blade balance. All changes were confirmed and no leftover old patterns remain except the exempted has_url amber badge.
outcome
All specified files (admin-string-hash-ban-button, admin-string-hash-index, admin-banned-string-index, admin-common-phrases-index-component, admin-texts-index, admin-text-show) have been updated with the a11y, badge, and button fixes per the worklist.
next steps
—
key decisions
- Used variant="info" for the violet flags (Brackets/Names/All UC) because they are informational flags, not admin-only.
- Used variant="danger" for Delete button (was subtle with red overrides).
- Used variant="subtle" for Unban (removed violet overrides).
- Left the has_url amber badge untouched as exempted per worklist.
- Converted sortable column headers from div to button, keeping all classes and wire:click.
- Used wire:loading.attr="disabled" and @disabled($submitted) on the ban button for proper disabled behavior.
open questions
—
1 week ago → 1 week ago