Skip to content

Clean app shell map offset overrides#417

Merged
ClarusIubar merged 1 commit into
mainfrom
app-shell-css-offset-cleanup
Jun 13, 2026
Merged

Clean app shell map offset overrides#417
ClarusIubar merged 1 commit into
mainfrom
app-shell-css-offset-cleanup

Conversation

@ClarusIubar

Copy link
Copy Markdown
Contributor

요약

  • 헤더/서브내비 grid 적용 후 남아 있던 legacy app-shell map offset override를 정리했습니다.
  • phone-shell__utility-slot dead CSS와 app-back-button/draggable back button 관련 CSS를 제거했습니다.
  • refinements.css가 더 이상 .map-filter-strip/.map-surface-frame!important로 덮지 않게 했습니다.
  • #409에서 늘어난 root hook count를 보정하기 위해 tourism map state hook을 owner-local src/hooks/map으로 이동했습니다.

아키텍처 근거

  • Responsibility map: index.css가 active app-shell map flow를 소유하고, refinements.css는 legacy visual refinements에만 남습니다.
  • Dependency direction: AppShell -> MapStage flow가 우선이며, refinements가 app-shell layout을 역으로 덮지 않습니다.
  • Test seam: source-quality unit test와 app-shell/critical UI E2E로 selector 회귀와 실제 배치를 검증했습니다.
  • Scope map: CSS offset cleanup과 hook locality 보정만 포함합니다. API/DB/OAuth/user copy 변경은 없습니다.
  • Architecture risk: refinements.css에는 다른 화면의 과거 !important가 남아 있으나 TSK-012-06 App shell CSS offset cleanup #410 범위의 map/back-button/utility overlap은 제거했습니다.

검증

  • npm.cmd run check:numeric-literals 통과
  • npm.cmd run lint 통과
  • npm.cmd run typecheck 통과
  • npm.cmd run test:unit 통과
  • npm.cmd run test:integration 통과
  • npm.cmd run test:regression 통과
  • npm.cmd run test:e2e 통과
  • npm.cmd run build 통과
  • git diff --check 통과
  • staged UTF-8/BOM 검사 통과

Closes #410

@ClarusIubar ClarusIubar merged commit fa67c88 into main Jun 13, 2026
6 checks passed
@ClarusIubar ClarusIubar deleted the app-shell-css-offset-cleanup branch June 13, 2026 02:17
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.

TSK-012-06 App shell CSS offset cleanup

1 participant