Skip to content

Fix: 모바일 드롭다운 메뉴 아이콘/라벨 겹침 (#160)#161

Merged
edwardkim merged 1 commit into
edwardkim:develfrom
seunghan91:fix/mobile-menu-icon-overlap
Apr 16, 2026
Merged

Fix: 모바일 드롭다운 메뉴 아이콘/라벨 겹침 (#160)#161
edwardkim merged 1 commit into
edwardkim:develfrom
seunghan91:fix/mobile-menu-icon-overlap

Conversation

@seunghan91

Copy link
Copy Markdown
Contributor

요약

모바일 뷰포트(≤767px)에서 드롭다운 메뉴의 아이콘이 라벨 텍스트와 겹치는 문제를 수정합니다.

Fixes #160

원인

rhwp-studio/src/styles/menu-bar.css의 기본 .md-itempadding: 5px 16px 5px 32pxposition: absolute; left: 8px; width: 16px 아이콘 자리를 왼쪽에 확보하고 있습니다.

rhwp-studio/src/styles/responsive.css의 모바일 미디어쿼리가 padding: 8px 16px(2-value shorthand)로 덮어쓰면서 왼쪽 여백이 32px → 16px로 축소되어, 라벨이 아이콘 영역과 8px 겹치게 됩니다.

수정

4-value padding으로 변경해 왼쪽 32px(아이콘 자리)를 유지합니다:

@media (width <= 767px) {
  .md-item {
    min-height: var(--touch-target-min);
-   padding: 8px 16px;
+   padding: 8px 16px 8px 32px;
    font-size: var(--font-size-md);
  }
}

터치 타겟 세로 8px 여백은 유지되며, 좌우 간격도 그대로입니다.

검증

  • 뷰포트 390×844 Chromium에서 재현 → 픽셀 단위 측정 확인 (아이콘 x=8–24, 라벨 x=16부터, 8px 겹침)
  • 인라인으로 fix CSS 주입 → 겹침 해소 확인
  • 데스크톱 뷰포트(≥768px)는 본 수정 영향 없음 (기본 규칙 그대로)
  • CSS-only 변경으로 cargo test / cargo clippy 영향 없음

스크린샷

Before/After 스크린샷을 PR 댓글로 추가합니다.

체크리스트

  • devel 브랜치를 대상으로 PR 생성
  • CSS-only 변경이므로 cargo 테스트 영향 없음
  • 데스크톱 렌더링에 회귀 없음 (미디어쿼리 ≤767px만 수정)

`.md-item`의 모바일 패딩이 2-value shorthand(`8px 16px`)로 덮어써져
왼쪽 여백이 32px → 16px로 축소되어, `position:absolute; left:8px`
아이콘이 라벨 텍스트와 약 8px 겹치는 문제 수정.

4-value로 변경하여 왼쪽 32px(아이콘 자리)를 유지하면서 세로 8px
터치 타겟 여백은 그대로 유지.

Fixes edwardkim#160
seunghan91 added a commit to seunghan91/rhwp that referenced this pull request Apr 16, 2026
@seunghan91

Copy link
Copy Markdown
Contributor Author

스크린샷 (iPhone 14, 390×844)

Before — "파일" 메뉴 열기 시 아이콘과 텍스트 겹침

before

"저장", "편집 용지", "인쇄", "제품 정보" 항목의 아이콘이 첫 글자(저/편/인/제) 위에 겹쳐 렌더링됩니다. 아이콘이 없는 "새로 만들기", "열기"는 정상입니다.

After — 이 PR 적용 후

after

아이콘과 라벨 텍스트가 정상적으로 분리됩니다.


스크린샷은 seunghan91/rhwpassets/pr-161-screenshots 브랜치에 별도로 보관했습니다 (PR diff에 포함되지 않음).

@edwardkim edwardkim left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

검증 완료 ✅

  • CSS 1줄 변경: padding: 8px 16pxpadding: 8px 16px 8px 32px
  • 원인 분석 정확: 모바일 미디어쿼리에서 2-value shorthand가 기본 스타일의 left padding(32px)을 덮어씀
  • 데스크톱 회귀 없음 (미디어쿼리 ≤767px 범위)

감사합니다! 👍

@edwardkim edwardkim merged commit 4c5406d into edwardkim:devel Apr 16, 2026
6 checks passed
edwardkim added a commit that referenced this pull request Apr 19, 2026
본 v0.5.0 → v0.7.3 (라이브러리) / 0.2.0 (확장) 배포 주기에
머지된 외부 기여자 6명을 README 3개 변경 이력에 추가:

- @ahnbu — Ctrl+S file handle (PR #189, 기명시)
- @bapdodi — 회전 도형 리사이즈 + Flip (PR #192)
- @dreamworker0 — Windows CFB 경로 (PR #152)
- @marsimon — HWP 그림 효과 SVG (PR #149)
- @postmelee — 썸네일 + options CSP (PR #168)
- @seunghan91 — HWPX Serializer + 다수 (PR #170, #161, #163, #153, #154)

각 README 끝에 "기여자 감사" 섹션 추가 — 6명 일괄 인정.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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.

2 participants