Skip to content

Conversation

@HopeFullee
Copy link
Member

📝 변경 사항

[문제]

  • 치영님이 모바일 Enter로 Tag 추가 안된다고 QA 해줌
  • 기존 keydown 이벤트의 code는 키보드의 물리적인 키 위치를 찾고 있음.
    하지만, 모바일 (IOS, Android 등)의 네이티브 가상 키보드는 물리적인 키 위치를 반환하지 않음. (당연하게도 가상 키보드 이기 때문)

[해결]

  • 물리적인 키 위치를 반환하는 event.code 대신 사용자가 입력한 키의 "의미"를 기준으로 반환하는 event.key를 적용함.

[수확]

  • keydown 이벤트의 code는 물리적인 키 위치를 반환함. 그래서 Mobile의 가상 키보드는 감지 할 수 없음.
  • keydown 이벤트의 key는 의미 (사용자가 누른 것이 Enter 인가? 등)를 기준으로 반환함.
  • 웹 개발은 PC/Mobile 등 크로스 브라우징을 신경써야 하는 분야이기 때문에 되도록이면 답정너 event.key 를 사용하는게 옳다고 봄. (뇌피셜)

🔗 관련 이슈

Closes #


🧪 테스트 방법

  • 수동 테스트 검증(로컬 환경)
  • 유닛 테스트 검증
  • 통합 테스트 검증

📸 스크린샷 (선택)


📋 체크리스트

  • 관련 문서를 업데이트했습니다 (필요한 경우)
  • 테스트를 추가/수정했습니다 (필요한 경우)
  • Breaking change가 있다면 명시했습니다

💬 추가 코멘트


CodeRabbit Review는 자동으로 실행되지 않습니다.

Review를 실행하려면 comment에 아래와 같이 작성해주세요

@coderabbitai review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 27, 2025

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch somang-fix/create-group

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@HopeFullee HopeFullee added the Ready For Review! 리뷰 받을 준비가 되었습니다. label Dec 27, 2025
@github-actions
Copy link

github-actions bot commented Dec 27, 2025

🎭 Playwright Report

E2E Test가 성공적으로 완료되었습니다.

Test 요약 내용을 확인해주세요.

Status Build Log Updated (UTC)
✅ Ready View Build 2025-12-27 15:36:18

📊 Test Summary

  • ✅ Passed: 3
  • ❌ Failed: 0
  • ⏱️ Duration: 25.2s

📜 Test Details

✅ Passed Tests (3)
  • profile.test.ts (3)
    • [chromium] 존재하지 않는 프로필 페이지로 접속 시 404 redirect 되는 지 테스트
    • [firefox] 존재하지 않는 프로필 페이지로 접속 시 404 redirect 되는 지 테스트
    • [webkit] 존재하지 않는 프로필 페이지로 접속 시 404 redirect 되는 지 테스트

@github-actions
Copy link

github-actions bot commented Dec 27, 2025

🎨 Storybook Report

변경 사항이 없습니다

모든 Story가 이전 빌드와 동일합니다.

Status Storybook Build Log Updated (UTC)
✅ Unchanged View Storybook View Build 2025-12-27 15:35:47

@github-actions
Copy link

github-actions bot commented Dec 27, 2025

🚀 PR Preview Report

Build가 성공적으로 완료되었습니다.

Preview에서 변경사항을 확인하세요.

Status Preview Build Log Updated (UTC)
✅ Ready Visit Preview View Logs 2025-12-27 15:36:35

@github-actions
Copy link

github-actions bot commented Dec 27, 2025

📊 Coverage Report

Status Build Log Updated (UTC)
✅ Ready View Build 2025-12-27 15:35:23

➡️ #260main에 병합하면 coverage가 0.00% 유지합니다.

Coverage 요약

@@             Coverage Diff             @@
##             main     #260       +/-   ##
===========================================
  Coverage   37.50%   37.50%     0.00%     
===========================================
  Files         189      189         0     
  Lines        8132     8132         0     
  Branches      361      361         0     
===========================================
  Hits         3050     3050         0     
  Misses       5082     5082         0     

영향받은 파일

이 PR로 영향받은 파일이 없습니다

수정된 모든 파일이 현재 coverage를 유지했습니다.

Copy link
Member

@claudia99503 claudia99503 left a comment

Choose a reason for hiding this comment

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

누워서 깃허브앱으로 승인 중 수고 하셨습니다~

@HopeFullee
Copy link
Member Author

주무시는데 죄송합니다..

@HopeFullee HopeFullee merged commit fcba332 into main Dec 27, 2025
8 checks passed
@HopeFullee HopeFullee deleted the somang-fix/create-group branch December 27, 2025 15:37
@claudia99503 claudia99503 changed the title fix: 모임생성 Tag 추가 시 Mobile 환경의 Enter도 감지하도록 event.code에서 event.key로 변경 [Fix] 모임생성 Tag 추가 시 Mobile 환경의 Enter도 감지하도록 event.code에서 event.key로 변경 Dec 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Ready For Review! 리뷰 받을 준비가 되었습니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants