style(admin): restyle admin user form to match global design system #14

Open
robot wants to merge 2 commits from bugfix/spider-ssr-gate into main
pull from: bugfix/spider-ssr-gate
Member

Restyle the admin user form SCSS to align with the global design system.

What Changed

Single file: src/app/admin/admin-users-tab.component.scss (+129, -11)

Changes

  • Alert banners — Added .form-error and .form-success classes using design-system tokens ($danger-red, $success-green, $spacing-sm, $spacing-md, $radius-md) for consistent error/success state styling
  • Section container — Replaced hardcoded .add-admin-section values (margin-top: 2rem, padding: 1.5rem, border: 1px solid var(--color-light), border-radius: 8px) with design-system variable equivalents
  • Form layout — Restyled card grid, form groups, button groups, and avatar upload area to use consistent spacing, typography, and color tokens from the shared variables and mixins SCSS modules
  • Responsive design — Added breakpoint-aware grid layouts and card hover states ($breakpoint-sm, $breakpoint-md)
  • Accessibility — Visible focus rings, .is-invalid red states on input errors, alert banners with semantic structure

What This PR Does NOT Include

The SSR/prerender flash fix and spider detection features that were previously planned for this branch have been completed and merged in earlier PRs (#11, #12). This PR contains only the admin form styling work.

Files Changed

Frontend (SCSS)

  • src/app/admin/admin-users-tab.component.scss — Full restyle using design-system tokens

Test Plan

  • Frontend builds successfully (frontend-build CI passed)
  • Frontend tests pass (frontend-test CI passed)
  • Visual verification: confirm admin user form renders correctly on desktop and mobile breakpoints
Restyle the admin user form SCSS to align with the global design system. ## What Changed Single file: `src/app/admin/admin-users-tab.component.scss` (+129, -11) ### Changes - **Alert banners** — Added `.form-error` and `.form-success` classes using design-system tokens (`$danger-red`, `$success-green`, `$spacing-sm`, `$spacing-md`, `$radius-md`) for consistent error/success state styling - **Section container** — Replaced hardcoded `.add-admin-section` values (`margin-top: 2rem`, `padding: 1.5rem`, `border: 1px solid var(--color-light)`, `border-radius: 8px`) with design-system variable equivalents - **Form layout** — Restyled card grid, form groups, button groups, and avatar upload area to use consistent spacing, typography, and color tokens from the shared `variables` and `mixins` SCSS modules - **Responsive design** — Added breakpoint-aware grid layouts and card hover states (`$breakpoint-sm`, `$breakpoint-md`) - **Accessibility** — Visible focus rings, `.is-invalid` red states on input errors, alert banners with semantic structure ### What This PR Does NOT Include The SSR/prerender flash fix and spider detection features that were previously planned for this branch have been completed and merged in earlier PRs (#11, #12). This PR contains only the admin form styling work. ## Files Changed **Frontend (SCSS)** - `src/app/admin/admin-users-tab.component.scss` — Full restyle using design-system tokens ## Test Plan - [x] Frontend builds successfully (`frontend-build` CI passed) - [x] Frontend tests pass (`frontend-test` CI passed) - [ ] Visual verification: confirm admin user form renders correctly on desktop and mobile breakpoints
robot added 1 commit 2026-08-09 04:01:58 -07:00
style(admin): restyle admin user form to match global design system
CI Pipeline / backend-test (push) Successful in 9m51s
CI Pipeline / frontend-test (push) Successful in 23s
CI Pipeline / frontend-build (push) Successful in 29s
CI Pipeline / backend-lint (push) Successful in 9s
CI Pipeline / backend-test (pull_request) Successful in 14m47s
CI Pipeline / frontend-test (pull_request) Successful in 24s
CI Pipeline / frontend-build (pull_request) Successful in 30s
CI Pipeline / backend-lint (pull_request) Successful in 10s
CI Pipeline / quality-gate (push) Successful in 3s
CI Pipeline / quality-gate (pull_request) Successful in 2s
cd6dc96214
- Replace hardcoded CSS custom properties with SCSS variables
- Add .form-error/.form-success alert banners
- Add .admin-form styles: .form-row grid, .form-group labels/inputs,
  focus rings, is-invalid states, placeholder styling
- Style .add-admin-section heading with $font-heading, divider underline
- Use button-style mixin for .btn-save (was unstyled before)
- Add .form-actions bar layout
- Add responsive breakpoint: stack form rows on mobile (<768px)
- Consistent spacing via $spacing-* tokens throughout
- Consistent border-radius via $radius-* tokens throughout
robot changed title from fix(ssr): resolve prerender flash and add spider detection + admin user form styling to style(admin): restyle admin user form to match global design system 2026-08-09 11:45:17 -07:00
kfj001 added 1 commit 2026-08-09 12:05:42 -07:00
Merge branch 'main' into bugfix/spider-ssr-gate
CI Pipeline / backend-test (push) Successful in 18m31s
CI Pipeline / frontend-test (push) Successful in 26s
CI Pipeline / frontend-build (push) Successful in 31s
CI Pipeline / backend-lint (push) Successful in 10s
CI Pipeline / backend-test (pull_request) Successful in 19m12s
CI Pipeline / frontend-test (pull_request) Successful in 25s
CI Pipeline / frontend-build (pull_request) Successful in 33s
CI Pipeline / backend-lint (pull_request) Successful in 11s
CI Pipeline / quality-gate (push) Successful in 3s
CI Pipeline / quality-gate (pull_request) Successful in 4s
9594187fc1
kfj001 approved these changes 2026-08-09 12:05:48 -07:00
All checks were successful
CI Pipeline / backend-test (push) Successful in 18m31s
CI Pipeline / frontend-test (push) Successful in 26s
CI Pipeline / frontend-build (push) Successful in 31s
CI Pipeline / backend-lint (push) Successful in 10s
CI Pipeline / backend-test (pull_request) Successful in 19m12s
CI Pipeline / frontend-test (pull_request) Successful in 25s
CI Pipeline / frontend-build (pull_request) Successful in 33s
CI Pipeline / backend-lint (pull_request) Successful in 11s
CI Pipeline / quality-gate (push) Successful in 3s
CI Pipeline / quality-gate (pull_request) Successful in 4s
You are not authorized to merge this pull request.
This pull request can be merged automatically.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin bugfix/spider-ssr-gate:bugfix/spider-ssr-gate
git checkout bugfix/spider-ssr-gate
Sign in to join this conversation.
No Reviewers
No labels
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: WesternTechnologies/kmtnflower#14