Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: 멤버 리스트 내 커피챗 뱃지 클릭시 멤버 카드 클릭과 함께 작동되는 문제 해결 #1643

Merged
merged 1 commit into from
Nov 3, 2024

Conversation

simeunseo
Copy link
Member

🤫 쉿, 나한테만 말해줘요. 이슈넘버

🧐 어떤 것을 변경했어요~?

커피챗 버튼 클릭시 멤버 카드의 link 동작을 막기 위해 e.preventDefault 추가

🤔 그렇다면, 어떻게 구현했어요~?

멤버 카드가 Link태그로 이루어져있어서, 그 내부에 있는 커피챗 버튼을 누를 때도 a태그의 href동작이 발생하게 됩니다.
이 동작을 방지하고 커피챗 상세로만 이동시키기 위해, 커피챗 버튼 onClick에 e.preventDefault()를 추가했습니다.

❤️‍🔥 당신이 생각하는 PR포인트, 내겐 매력포인트.

📸 스크린샷, 없으면 이것 참,, 섭섭한데요?

AS-IS

2024-11-03.12.00.53.mov

TO-BE

2024-11-03.12.01.12.mov

Copy link

height bot commented Nov 3, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

Copy link

github-actions bot commented Nov 3, 2024

✨✨ 스토리북으로 확인하기 ✨✨

Copy link

github-actions bot commented Nov 3, 2024

🚀 프리뷰 배포 확인하기 🚀

https://cdbdb318.sopt-internal-dev.pages.dev

@simeunseo simeunseo self-assigned this Nov 3, 2024
@simeunseo simeunseo added the 🐛 Bug 버그픽스 label Nov 3, 2024
@simeunseo simeunseo added this pull request to the merge queue Nov 3, 2024
Merged via the queue into main with commit f6ef60b Nov 3, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix: 멤버 리스트 내 커피챗 뱃지 클릭시 멤버 카드 클릭과 함께 작동되는 문제 해결
2 participants