Skip to content

Conversation

@cccwon2
Copy link
Contributor

@cccwon2 cccwon2 commented Dec 14, 2024

주요 변경사항

  1. FilterDropdown
  • handleSelect 함수에서 null 체크 추가
  • 버튼 클릭 이벤트에 preventDefault 추가
  • 외부 클릭 시 드롭다운만 닫히도록 수정
  1. DropdownList
  • DropdownItem의 클릭 핸들러에 stopPropagation 추가
  • 이벤트 전파 방지 로직 강화
  1. InputDropdown
  • handleInputClick 함수 추가
  • input의 onClick 이벤트에 handleInputClick 연결
  • 이벤트 전파 방지와 함께 드롭다운 직접 열기
  1. TimePickerInput
  • seCallback 추가하여 함수 메모이제이션
  • cn 유틸리티 추가 및 사용
  • readOnly={true}를 readOnly로 단순화
  • 불필요한 !important 제거
  • 이벤트 핸들러 의존성 배열 추가
  • 클래스네임 문자열 정리
  1. useDropdownOpen
  • Dispatch와 SetStateAction 타입 import 추가
  • setIsOpen 호출 시 함수형 업데이트 사용
  • 타입 명시적 지정 (prev: boolean)
  1. SearchSection
  • pathname 추가 및 연동

수정 후 해결된 문제

  1. 더블 클릭 시 빈 메뉴가 생기는 현상이 해결됨
  2. 이벤트 전파가 적절히 제어됨
  3. 드롭다운이 더 안정적으로 동작함

@cccwon2 cccwon2 added the 🐞 Fix 버그 수정에 대한 커밋 label Dec 14, 2024
@cccwon2 cccwon2 self-assigned this Dec 14, 2024
@github-actions
Copy link

github-actions bot commented Dec 14, 2024

💄 Storybook: https://673dcf13c9a3a1189a7c6fd4-bsmvzdlrry.chromatic.com/ # Chromatic에서 생성된 Storybook URL
🕖 Update: 2024년 12월 14일 21시 14분 12초 # 현재 시간

Copy link
Collaborator

@hongggyelim hongggyelim left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 👍

@cccwon2 cccwon2 added the 📦️ Chore 자잘한 수정 label Dec 14, 2024
@yyezzzy yyezzzy removed their request for review December 14, 2024 12:26
Copy link
Collaborator

@yyezzzy yyezzzy left a comment

Choose a reason for hiding this comment

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

금방 수정해주셨군여 👍

@yyezzzy yyezzzy merged commit f2dd435 into dev Dec 14, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📦️ Chore 자잘한 수정 🐞 Fix 버그 수정에 대한 커밋

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants