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

Bug: 친구추가 및 삭제 버튼 attribute 잘 받아오도록 수정 #116

Merged
merged 1 commit into from
Jul 1, 2024

Conversation

mixsung
Copy link
Contributor

@mixsung mixsung commented Jun 30, 2024

#️⃣연관된 이슈

#115

📝작업 내용

프로필에서 친구 삭제, 추가 버튼을 클릭했을 때 attribute를 받아오지 못하는 버그를 수정했습니다.

  • 원인 1. div에 eventListener를 걸었고 attribute는 button에 있음

    • button에 eventListener 걸고 attribute는 button에 유지
  • 원인 2. div와 button의 크기가 다름. div 안에 button이 중앙에 배치되어있어 div, button 둘 다 클릭할 수 있는 상황

    • div는 오직 크기 조절용으로 css 설정 조정, button에서 거의 모든 css 설정 조정. button의 width 100%로 button이 div를 덮게 만듦

추가로 tab 이동 및 엔터 이벤트도 다시 살펴봤습니다.

스크린샷 (선택)

스크린샷 2024-06-30 오후 9 30 29 스크린샷 2024-06-30 오후 9 52 17

💬리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

하나 고치면 하나 안돼서 git restore 4번 했는데 이게 맞나요

@mixsung mixsung added Bug Something isn't working Refactor Extra attention is needed labels Jun 30, 2024
@mixsung mixsung added this to the 6월 4주차 milestone Jun 30, 2024
@mixsung mixsung requested a review from Hosung99 June 30, 2024 15:06
@mixsung mixsung self-assigned this Jun 30, 2024
Copy link
Member

@Hosung99 Hosung99 left a comment

Choose a reason for hiding this comment

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

로직 깔끔하게 잘 짜신 것 같습니다.
고생하셨어요~!

@Hosung99 Hosung99 merged commit da84763 into dev Jul 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Refactor Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants