erik/slugkit
1
0
Fork 0
Code Issues 6 Pull requests Projects Releases 6 Packages Wiki Activity Actions

Redesign admin pages layout #164

Merged
erik merged 2 commits from feat/task-8ad38b86-admin-layout into main 2026-06-20 08:21:40 -05:00
Conversation 2 Commits 2 Files changed 9 +786 -524
erik commented 2026-06-20 08:05:06 -05:00
Owner
Copy link

Summary

  • Add a shared Tailwind-powered admin layout shell with sidebar navigation, signed-in panel, logout, stylesheet loading, and color-mode toggle support.
  • Redesign login, admin dashboard, users, passkeys, API keys, and ActivityPub pages with consistent cards, forms, buttons, tables, badges, notices, and dark-mode classes.
  • Rebuild generated Tailwind CSS for the new admin utility classes.
  • Extend auth route assertions for the shared stylesheet, admin navigation, and dark-mode support.

Verification

  • npm run build --workspace @slugkit/template-site
  • Targeted template-site admin/auth route tests
  • ./scripts/pre-pr.sh
## Summary - Add a shared Tailwind-powered admin layout shell with sidebar navigation, signed-in panel, logout, stylesheet loading, and color-mode toggle support. - Redesign login, admin dashboard, users, passkeys, API keys, and ActivityPub pages with consistent cards, forms, buttons, tables, badges, notices, and dark-mode classes. - Rebuild generated Tailwind CSS for the new admin utility classes. - Extend auth route assertions for the shared stylesheet, admin navigation, and dark-mode support. ## Verification - `npm run build --workspace @slugkit/template-site` - Targeted template-site admin/auth route tests - `./scripts/pre-pr.sh`
feat: redesign admin layout
All checks were successful
CI / build-lint-test (pull_request) Successful in 48s
Details
324d7dacbc 
Task: #task-8ad38b86
erik commented 2026-06-20 08:06:32 -05:00
Author
Owner
Copy link

PR Review: Approved

Summary

This PR replaces the previous standalone inline-styled admin pages with a shared Tailwind-powered admin shell and consistent settings-style components. The redesign covers login, dashboard, site users, passkeys, API keys, and ActivityPub settings, adds shared sidebar navigation and color-mode controls, and updates generated Tailwind CSS plus route assertions for stylesheet/navigation/dark-mode presence.

Acceptance criteria

  • Admin pages use Tailwind CSS styling consistently — met: admin templates now use shared Tailwind class constants from admin-layout.ts, link /styles/public.css, and remove page-local inline style blocks.
  • Admin pages have a clear admin/settings-style layout — met: authenticated admin pages now share a sidebar, active navigation, signed-in panel, logout, page headers, cards, tables, badges, and structured settings sections.
  • Layout is easier to navigate and use — met: dashboard cards link to each admin area, sidebar navigation is present on all admin pages, and forms/tables/actions are grouped consistently.
  • Styling matches the rest of the site — met: admin pages use the same Tailwind palette, rounded cards, shadows, teal accents, gray surfaces, and generated public stylesheet as the public UI.
  • Light and dark mode both render correctly — met: admin documents include the color-mode script/toggle, meta color-scheme, and light/dark Tailwind classes; route assertions cover stylesheet, toggle, and dark-mode classes.

Blocking issues

None.

Warnings

None.

Verification

  • npm run build --workspace @slugkit/template-site
  • Targeted template-site admin/auth route tests
  • ./scripts/pre-pr.sh
  • Forgejo Actions run #229 passed.

Verdict

Approved. Stop at the human merge gate.

PR Review Worker Result

  • pr: open
  • review: approved
  • pr_comment: posted
  • task_comment: posted
  • blocking_issues: 0
  • warnings: 0
  • criteria_total: 5
  • criteria_met: 5
  • criteria_missing: 0
## PR Review: Approved ### Summary This PR replaces the previous standalone inline-styled admin pages with a shared Tailwind-powered admin shell and consistent settings-style components. The redesign covers login, dashboard, site users, passkeys, API keys, and ActivityPub settings, adds shared sidebar navigation and color-mode controls, and updates generated Tailwind CSS plus route assertions for stylesheet/navigation/dark-mode presence. ### Acceptance criteria - [x] Admin pages use Tailwind CSS styling consistently — met: admin templates now use shared Tailwind class constants from `admin-layout.ts`, link `/styles/public.css`, and remove page-local inline style blocks. - [x] Admin pages have a clear admin/settings-style layout — met: authenticated admin pages now share a sidebar, active navigation, signed-in panel, logout, page headers, cards, tables, badges, and structured settings sections. - [x] Layout is easier to navigate and use — met: dashboard cards link to each admin area, sidebar navigation is present on all admin pages, and forms/tables/actions are grouped consistently. - [x] Styling matches the rest of the site — met: admin pages use the same Tailwind palette, rounded cards, shadows, teal accents, gray surfaces, and generated public stylesheet as the public UI. - [x] Light and dark mode both render correctly — met: admin documents include the color-mode script/toggle, `meta color-scheme`, and light/dark Tailwind classes; route assertions cover stylesheet, toggle, and dark-mode classes. ### Blocking issues None. ### Warnings None. ### Verification - `npm run build --workspace @slugkit/template-site` - Targeted template-site admin/auth route tests - `./scripts/pre-pr.sh` - Forgejo Actions run #229 passed. ### Verdict Approved. Stop at the human merge gate. PR Review Worker Result - pr: open - review: approved - pr_comment: posted - task_comment: posted - blocking_issues: 0 - warnings: 0 - criteria_total: 5 - criteria_met: 5 - criteria_missing: 0
fix: align admin header and navigation
All checks were successful
CI / build-lint-test (pull_request) Successful in 44s
Details
2ad9b7c262 
Task: #task-8ad38b86
erik commented 2026-06-20 08:21:32 -05:00
Author
Owner
Copy link

PR Review Update: Approved

Summary

Reviewed the follow-up changes after the initial approval. The admin layout now keeps the same public site header shape without invented public navigation links, admin navigation order is Overview, Passkeys, API keys, Site users, ActivityPub, and viewer denial for ActivityPub management is explicitly covered by tests.

Acceptance criteria

  • Admin pages use Tailwind CSS styling consistently — met.
  • Admin pages have a clear admin/settings-style layout — met.
  • Layout is easier to navigate and use — met, with the requested admin navigation order.
  • Styling matches the rest of the site — met, including the main site header treatment.
  • Light and dark mode both render correctly — met, with shared stylesheet/color-mode support preserved.

Blocking issues

None.

Warnings

None.

Verification

  • ./scripts/pre-pr.sh
  • Forgejo Actions run #230 passed.

Verdict

Approved. Human merge approval has been provided.

PR Review Worker Result

  • pr: open
  • review: approved
  • pr_comment: posted
  • task_comment: posted
  • blocking_issues: 0
  • warnings: 0
  • criteria_total: 5
  • criteria_met: 5
  • criteria_missing: 0
## PR Review Update: Approved ### Summary Reviewed the follow-up changes after the initial approval. The admin layout now keeps the same public site header shape without invented public navigation links, admin navigation order is Overview, Passkeys, API keys, Site users, ActivityPub, and viewer denial for ActivityPub management is explicitly covered by tests. ### Acceptance criteria - [x] Admin pages use Tailwind CSS styling consistently — met. - [x] Admin pages have a clear admin/settings-style layout — met. - [x] Layout is easier to navigate and use — met, with the requested admin navigation order. - [x] Styling matches the rest of the site — met, including the main site header treatment. - [x] Light and dark mode both render correctly — met, with shared stylesheet/color-mode support preserved. ### Blocking issues None. ### Warnings None. ### Verification - `./scripts/pre-pr.sh` - Forgejo Actions run #230 passed. ### Verdict Approved. Human merge approval has been provided. PR Review Worker Result - pr: open - review: approved - pr_comment: posted - task_comment: posted - blocking_issues: 0 - warnings: 0 - criteria_total: 5 - criteria_met: 5 - criteria_missing: 0
erik merged commit b786af01f8 into main 2026-06-20 08:21:40 -05:00
erik deleted branch feat/task-8ad38b86-admin-layout 2026-06-20 08:21:40 -05:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
activitypub

   
admin

   
api

   
articles

   
auth

   
bug

   
cleanup

   
cli

   
comments

   
compatibility

   
config

   
contacts

   
database

   
deployment

   
design

   
dev-env

   
docs

   
documentation

   
email

   
enhancement

   
feature

   
federation

   
feed

   
homepage

   
implementation

   
integration

   
media

   
openapi

   
priority:high

   
priority:low

   
priority:medium

   
proof

   
public-routes

   
public-ui

   
release

   
safety

   
social

   
sources

   
status:active

   
status:canceled

   
status:done

   
status:inprogress

   
status:waiting

   
syndication

   
tailwind

   
template

   
test

   
web
No labels
activitypub
admin
api
articles
auth
bug
cleanup
cli
comments
compatibility
config
contacts
database
deployment
design
dev-env
docs
documentation
email
enhancement
feature
federation
feed
homepage
implementation
integration
media
openapi
priority:high
priority:low
priority:medium
proof
public-routes
public-ui
release
safety
social
sources
status:active
status:canceled
status:done
status:inprogress
status:waiting
syndication
tailwind
template
test
web
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
erik/slugkit!164
No description provided.