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

[FE][Feat] #108 : Dropdown 버튼 구현 #153

Merged
merged 6 commits into from
Nov 13, 2024
Merged

Conversation

effozen
Copy link
Collaborator

@effozen effozen commented Nov 13, 2024

📝 PR 개요

  • Dropdown button 구현

🔍 변경 사항

Dropdown 버튼을 구현했고, 사용처가 2개정도로 파악이 되어서 이런 장소들에 대응될 수 있도록 최대한 범용적으로 구성하였습니다.

✅ 체크리스트 (Checklist)

  • 코드가 빌드 오류 없이 잘 작동하는지 확인
  • 테스트가 통과하는지 확인 -> 테스트코드는 추후 작성 예정
  • 스타일 가이드와 일관성을 유지했는지 확인 -> 조금 더 수정할 예정입니다.
  • 관련 문서가 업데이트되었는지 확인 (선택 사항)
  • 리뷰어가 이해할 수 있도록 주석이나 설명을 추가했는지 확인

🏃 Todos

1가지 이슈가 있습니다.

  1. component가 unmount 되었을 때의 애니메이션 적용 안되는 문제 (우선은 뺴두었습니다.)

📷 스크린샷 및 동영상 (선택 사항)

2024-11-13.9.45.45.mov

📚 참고 자료 (선택 사항)

@github-actions github-actions bot added the 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) label Nov 13, 2024
juwon5272
juwon5272 previously approved these changes Nov 13, 2024
@juwon5272
Copy link
Collaborator

컴포넌트가 세세히 잘 나누어져 있어서 좋은거 같습니다. Todo도 적혀있어서 다음 작업을 파악하기 쉬웠구요

Todos:
- unmount 이미지 구현
- 아이콘 색상 들어가는 요소 수정 필요
- 링크 붙이기
@effozen effozen requested review from juwon5272 and removed request for juwon5272 November 13, 2024 11:02
Copy link
Collaborator Author

@effozen effozen left a comment

Choose a reason for hiding this comment

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

확인하였습니다.

@effozen effozen requested a review from juwon5272 November 13, 2024 11:03
@effozen effozen merged commit 2afef59 into frontend Nov 13, 2024
3 of 7 checks passed
@effozen effozen deleted the feature/fe/#108-drawer branch November 16, 2024 21:07
@effozen effozen added 작업 완료 PR이 제대로 되어 approve 된 상태 (PR 받은 두명 중 마지막 사람이 상태 변경) and removed 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) labels Nov 17, 2024
@github-actions github-actions bot added the 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) label Nov 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
작업 완료 PR이 제대로 되어 approve 된 상태 (PR 받은 두명 중 마지막 사람이 상태 변경) 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants