feat(web): migrate admin surface off bespoke CSS (migration phase 3, surface 3) - #791
Conversation
…surface 3) Re-plumbs /admin/* (workspaces, users, email, metrics, oauth + oauth detail) onto Tailwind utilities via shared class-string constants in admin-ui.ts (ADMIN_TH/ADMIN_TD/ADMIN_BTN/etc). Legacy class names stay in markup as hooks. admin.css/admin-workspaces.css/admin-metrics.css/admin-oauth.css reduced from 889 to 200 lines, keeping only data-state colors, JS-toggled .is-open/:empty structural rules, the responsive column collapse, and the metrics inline-SVG chart (interactive hover/tooltip).
|
|
Maintained by uploads.sh · add media: |
|
Important Review skippedAuto reviews are limited based on label configuration. 🏷️ Required labels (at least one) (2)
🚫 Excluded labels (none allowed) (1)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |



Third phase-3 surface of the Tailwind + shadcn migration (follows #787/#789; plan in
.context/2026-08-22-tailwind-shadcn-migration-plan.md, local doc).What
admin-ui.ts(table cells, buttons, mini-lists) keep the repeated chrome DRY across pages.[data-state]status colors, JS-toggled.is-openrules, the 680px column collapse, the inline-SVG metrics chart styling, and theoauth-scopesfieldset.ul-select(width:100%, unlayered) use inlinestyle="width:auto"rather than a utility that would silently lose.Verification
astro build, lint,admin-uivitest 3/3 green.bg-panel/55replacing acolor-mix()(visually close, not bit-identical), and the expand panel's nested list spacing is authored per-list now.Screenshots attach via the managed comment.