Skip to content

Conversation

@sohyun0
Copy link
Contributor

@sohyun0 sohyun0 commented Oct 8, 2025

📝 작업 개요 (필수)

  1. useDropdown.ts 사용되지 않는 코드 정리
  2. useDropdownPosition.ts 임계값 하드코딩에서 파라미터로 받아서 사용하는 방법으로 변경
  3. dropdown 컨트롤영역 분리 및 오타수정
  4. icon aria-label 옵셔널로 변경

✨ 작업 내용 (필수)

  • 기능 구현
  • 버그 수정
  • 스타일/UI 변경
  • 리팩토링
  • 최적화/성능개선
  • 문서 업데이트
  • 기타 변경사항

📸 스크린샷

🧐 해결해야 하는 문제

🤔 리뷰어 확인 필요 사항

icon aria-label 옵셔널로 변경해서 decorative 인 경우에는 작성을 따로 안해도 되게끔 했습니다.
decorative 이 아닌경우에는 옵셔널이라 타입에러는 발생하지 않겠지만 접근성 면에서 중요하니 작성해주시면 될것 같습니다~!

🔗 관련 이슈

🛠️ 후속 작업

  • 추후 리팩토링 요소 발생시 이어가겠습니다.
  • [ ]

✅ 체크리스트 (필수)

  • 작업한 내용과 커밋 메시지 컨벤션을 통일했는지 확인
  • 내가 작성한 코드를 테스트까지 완료했는지 잘 작동했는지 확인
  • ESLint 검사 통과
  • Prettier 포맷팅 적용
  • TypeScript 에러 없음
  • 빌드 에러 없음

@sohyun0 sohyun0 self-assigned this Oct 8, 2025
@sohyun0 sohyun0 added ✨ feat 새로운 기능 추가 ♻️ refactor 결과 변화 없이 코드 구조 개선 🔧 chore 그 외 기타 수정 (빌드 스크립트 수정, 패키지 관리, 파일 삭제/이름 변경 등) labels Oct 8, 2025
@vercel
Copy link

vercel bot commented Oct 8, 2025

@sohyun0 is attempting to deploy a commit to the projects Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Oct 8, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
thejulge Ready Ready Preview Comment Oct 8, 2025 7:19pm

@BaeZzi813 BaeZzi813 merged commit 6edfcbd into codeit-FE18-part3:develop Oct 9, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔧 chore 그 외 기타 수정 (빌드 스크립트 수정, 패키지 관리, 파일 삭제/이름 변경 등) ✨ feat 새로운 기능 추가 ♻️ refactor 결과 변화 없이 코드 구조 개선

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] Dropdown 컴포넌트 리팩토링

2 participants