Skip to content

Conversation

@hyeonjin6530
Copy link
Member

📌 관련 이슈번호

🎟️ PR 유형

어떤 변경 사항이 있나요?

  • 새 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 리팩토링

Check List

  • 🔀 PR 제목의 형식을 잘 작성했나요?
  • 🏗️ yarn build는 성공했나요?
  • 🧹 불필요한 코드는 제거했나요? (TODO, 주석, clg... etc.)
  • ✅ 컨벤션을 지켰나요?
  • 💭 이슈는 등록했나요?
  • 🏷️ 라벨은 등록했나요?
  • 🙇‍♂️ 리뷰어를 지정했나요?

✅ Key Changes

이번 PR에서 작업한 내용을 간략히 설명해주세요

  1. 작업 내용
  • 모바일 메인 검색창 클릭시 확대되는 현상을 수정
  • 알림이 깨지는 현상을 수정
  • 알림이 3자리가 넘어갔을 때 99+로 표시되도록 로직 추가

📢 To Reviewers

  • 리다이렉트되는 주소가 localhost라서 모바일로 테스트를 할 때 ip주소로 하려면 좀 강제로 주소를 바꿔서 테스트 해야하더라구용 테스트할 때 참고하시면 될 거 같습니다!!☺️

📸 스크린샷

알림이 99개를 넘어갔을 때
image

🔗 참고 자료

@vercel
Copy link

vercel bot commented Mar 3, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 3, 2025 6:10am

@github-actions github-actions bot requested a review from sinji2102 March 3, 2025 06:10
@github-actions github-actions bot added 🐛 BUG 버그를 고친 경우 ✨ FEAT 기능 구현 labels Mar 3, 2025
@hyeonjin6530 hyeonjin6530 added ✨ FEAT 기능 구현 and removed ✨ FEAT 기능 구현 labels Mar 3, 2025
{notificationCount && (
<div className="absolute -right-0.5 -top-0.5 flex h-[13px] w-[13px] items-center justify-center rounded-full bg-warning text-caption-2_midi text-white-primary">
{notificationCount}
{notificationCount > 0 && (
Copy link
Member

Choose a reason for hiding this comment

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

P5) 알림이 0개일 때 숫자 0만 뜨는 오류가 있었는데, 이렇게 하면 같이 해결될 것 같아용 좋습니당 🫶

Copy link
Member Author

Choose a reason for hiding this comment

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

히히 감사해용

Comment on lines +33 to +35
<meta
name="viewport"
content="initial-scale=1.0,user-scalable=no,maximum-scale=1,width=device-width"
Copy link
Member

Choose a reason for hiding this comment

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

P5) 최고예용 굿!!

Copy link
Member

@sinji2102 sinji2102 left a comment

Choose a reason for hiding this comment

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

아이폰에서 14px였나? 이하로 인풋을 넣으면 확대되는 오류가 있던 것 같은데 메타태그로 해결되는 오류였는지 헷갈리네용... 빠르게 머지해서 확인해보면 좋을 것 같아요!!

@hyeonjin6530 hyeonjin6530 merged commit 34ea376 into develop Mar 3, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 BUG 버그를 고친 경우 ✨ FEAT 기능 구현

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ Bug ] 모바일 메인 검색창 클릭시 확대되는 현상 / 알림 디자인 및 로직 수정

3 participants