Repository navigation
[fix] 터치 기기에서 탭한 뒤 hover 효과가 남지 않게 한다 - #2255
Conversation
- 터치 기기는 hover가 없어서 브라우저가 첫 탭에 :hover를 적용하고 다른 곳을 탭할 때까지 유지하므로, hover에 건 transform이 탭한 뒤에도 남는다 - FloatingButtonGroup·ClubFeed·RelatedPromotionCard·AdminContactFloatingButton·RecruitPage 화살표의 hover를 ClubCard와 같은 @media (hover: hover)로 감싼다
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Hover 스타일에 미디어 조건 적용 frontend/src/components/common/FloatingButtonGroup/FloatingButtonGroup.styles.ts, frontend/src/pages/AdminPage/components/AdminContactFloatingButton/AdminContactFloatingButton.styles.ts, frontend/src/pages/ClubDetailPage/components/ClubFeed/ClubFeed.styles.ts, frontend/src/pages/PromotionPage/components/detail/RelatedPromotionSection/RelatedPromotionCard/RelatedPromotionCard.styles.ts, frontend/src/pages/RecruitPage/RecruitPage.styles.ts |
5개 컴포넌트의 hover 스타일을 @media (hover: hover) 조건으로 제한했습니다. SVG 확대 배율, 사진 항목 효과, 카드 이동 거리, 셰브론 이동 거리는 기존 값과 같습니다. |
Priority:
Estimated code review effort: 2 (Simple) | ~10 minutes
Change: Bug fix · Severity of issue fixed: Low
Suggested labels: 💻 FE
Suggested reviewers: suhyun113
Merge Risk: 
Minimal · up to 290e9
-
No actionable merge-blocking risk remains.
Pre-merge checks: 
5
-
Full details: Linked Issues check
-
Explanation
직접 연결된 활성 이슈 [
#2254]의 구현 요구사항을 충족합니다. 5개 지정 위치의:hover규칙을@media (hover: hover)안으로 이동했습니다. 변경된 변형 값은 유지되므로 hover 지원 환경의 동작을 유지하고,hover: none환경의 hover 변형 적용을 제한합니다. 변경 diff는FloatingButtonGroup,ClubFeed,RelatedPromotionCard,AdminContactFloatingButton,RecruitPage에만 해당합니다. 이슈의 브라우저 대조 항목은 비코딩 검증 작업이므로 미실기기 검증을 코딩 요구사항 미충족으로 처리하지 않습니다.
Full details: Docstring Coverage
-
Explanation
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 5 files.
-
Finishing Touches
-
📝 Generate docstrings
-
- Commit to this branch
- Create a new PR
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands.
|
Claude finished @seongwon030's task in 23s —— View job harry 리뷰 총평리뷰 기준에 걸리는 곳이 없어서 통과예요 👍 이미
|
🎨 UI 변경사항을 확인해주세요
3개 스토리 변경 · 전체 193개 스토리 · 67개 컴포넌트 |
Close #2254
문제
모바일 웹과 앱 웹뷰에서 일부 버튼과 카드가 탭한 뒤에도 떠오르거나 커진 채로 남는다. 터치 기기에는 hover가 없어서 브라우저가 첫 탭에
:hover를 적용하고, 다른 곳을 탭할 때까지 그대로 둔다.&:hover85곳 중@media (hover: hover)로 감싼 곳은ClubCard하나뿐이다.해결
위치나 크기가 바뀌어 눈에 띄는 5곳의 hover 블록을
ClubCard와 같은@media (hover: hover)로 감쌌다.FloatingButtonGroup아이콘scale(1.12)ClubFeed사진translateY(-4px),RelatedPromotionCardtranslateY(-2px)AdminContactFloatingButtontranslateY(-2px),RecruitPage화살표translateX(3px)검증
Playwright에서
:hover를 강제(CDPforcePseudoState)하고FloatingButtonGroup공유 버튼 svg의transform을 전후로 비교했다.hover: hover)hover: none)scale(1.12)scale(1.12)scale(1.12)nonetypecheck, 관련 jest 8개 스위트(39개 테스트) 통과FloatingButtonGroup하나다. 나머지 4곳은 같은 치환이라 diff로만 확인했다영향 범위
학생용(메인 플로팅 버튼, 동아리 상세 피드, 홍보 상세, 모집 페이지), 관리자(문의 플로팅 버튼), 앱 웹뷰에 닿는다. 데스크톱 마우스 hover는 그대로다. 터치에서
ClubFeed·RelatedPromotionCard·RecruitPage화살표는:active가 없어서 누를 때 보이는 반응이 없어진다. 셋 다 누르면 바로 모달이 열리거나 페이지가 바뀐다.한계와 대안(선택)
maximum-scale=1, user-scalable=no는 입력창 글자 크기를 디자인과 정한 뒤 별도 PR로 다룬다Summary by CodeRabbit