Skip to content

fix(web): 모바일 네비게이션 메뉴 + 부트스트랩 401 레이스 제거 - #231

Merged
i3months merged 1 commit into
devfrom
fix/mobile-nav-and-auth-race
Sep 2, 2026
Merged

fix(web): 모바일 네비게이션 메뉴 + 부트스트랩 401 레이스 제거#231
i3months merged 1 commit into
devfrom
fix/mobile-nav-and-auth-race

Conversation

@i3months

@i3months i3months commented Sep 2, 2026

Copy link
Copy Markdown
Member

변경 사항

프론트엔드 점검에서 발견한 두 가지를 수정.

1) 모바일 네비게이션 갭

SiteNav 에 햄버거 메뉴가 없어, md 미만에서 마케팅 네비(기능·동작 방식·FAQ)와 (로그인 사용자의) 워크스페이스 링크가 헤더에서 사라졌다.

  • 햄버거 토글 + 드롭다운 메뉴 추가 → 해당 항목 수용
  • 시작하기 CTA 는 그대로 노출 유지(전환 동선 보존)
  • 접근성: aria-expanded/aria-controls, Esc 닫기, 링크 클릭 시 닫기
  • 390px 로컬 빌드 렌더로 열림/토글/링크 노출 확인

2) 부트스트랩 401 레이스

앱 로드 시 AuthProvider 가 토큰 없이 /api/users/me 를 먼저 쏴 401 → refresh → 재시도하던 낭비가 있었다(매 로드 401 1건 + 백엔드 WARN 로그 노이즈).

  • client.tsensureAccessToken() 추가 → refresh 로 토큰을 먼저 확보한 뒤 사용자 정보 호출
  • 세션이 없으면 refresh 401 로 비로그인 처리(동작 동일, 불필요한 401 제거)

typecheck·lint·build·vitest(182) 전부 통과.

1) 모바일 네비 갭
   md 미만에서 마케팅 네비(기능·동작 방식·FAQ)와 (로그인 사용자의) 워크스페이스
   링크가 헤더에서 사라졌다. 햄버거 토글 + 드롭다운 메뉴를 추가해 해당 항목을
   담았다. 시작하기 CTA 는 그대로 노출 유지. aria-expanded/controls, Esc 닫기,
   링크 클릭 시 닫기까지 접근성 반영.

2) 부트스트랩 401 레이스
   앱 로드 시 AuthProvider 가 토큰 없이 /api/users/me 를 먼저 쏴 401 → refresh
   → 재시도하던 낭비가 있었다(매 로드 401 1건, 백엔드 WARN 로그 노이즈). client
   에 ensureAccessToken() 을 추가해 refresh 로 토큰을 먼저 확보한 뒤 사용자 정보를
   부르도록 부트스트랩을 변경. 세션이 없으면 refresh 401 로 비로그인 처리.

typecheck·lint·build·vitest(182) 전부 통과. 모바일 메뉴 렌더/토글은 로컬 빌드로 확인.
@i3months
i3months merged commit e34d203 into dev Sep 2, 2026
5 checks passed
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