Skip to content

feat(security-ui): security domain local nav as left LNB sidebar (shell parity)#945

Open
seonghobae wants to merge 1 commit into
developfrom
feat/security-layout-left-lnb
Open

feat(security-ui): security domain local nav as left LNB sidebar (shell parity)#945
seonghobae wants to merge 1 commit into
developfrom
feat/security-layout-left-lnb

Conversation

@seonghobae

Copy link
Copy Markdown
Contributor

Why

Canonical shell (docs/ui-ux/naruon-ui-ux-mapping.md §4.1 + mockup_06/mockup_24) = top GNB + left LOCAL LNB + work area. SecurityLayout rendered its areas (보안 대시보드/접근 권한/감사 로그/외부 공유/정책) as a horizontal top-tab strip.

What

  • Desktop (lg+): left <nav aria-label="보안 로컬 탐색"> sidebar — domain title + five areas as vertical nav buttons (aria-current active).
  • Mobile (<lg): collapses to the existing tab strip — unchanged.

Verification (local)

  • src/app/security/page.test.tsx: 3 passed; tsc + eslint clean.
  • Runtime (1440px): GNB + left LNB (보안과 관리자: 보안 대시보드/접근 권한[active]/감사 로그/외부 공유/정책) + content. ✅

Scope

Second per-domain LNB conversion (after #944 Data). Remaining: Settings / Mail / AI Hub / Projects / Tasks.

🤖 Generated with Claude Code

…ebar

Same shell-parity fix as the data domain (docs/ui-ux/naruon-ui-ux-mapping.md
§4.1 + mockup_06/24): security rendered its areas (보안 대시보드/접근 권한/
감사 로그/외부 공유/정책) as a horizontal top-tab strip; the canonical shell
uses a left LOCAL LNB sidebar.

- Desktop (lg+): left <nav aria-label='보안 로컬 탐색'> sidebar with the domain
  title + the five areas as vertical nav buttons (aria-current on active).
- Mobile (<lg): collapses to the existing horizontal tab strip; unchanged.

Verified: security page tests 3 passed, tsc + eslint clean, 1440px runtime
capture shows GNB + left LNB + content.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017RkKdtHRLG4wSLh6PVsp8J
@seonghobae seonghobae enabled auto-merge July 6, 2026 23:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant