Skip to content

Conversation

@SSUK-H
Copy link
Collaborator

@SSUK-H SSUK-H commented Feb 21, 2025

📝작업 내용

  • 여행 정보 드롭다운 메뉴 버튼 중앙이 안눌리는 에러 해결
  • 드롭다운 컴포넌트 유닛 테스트

✨PR Point

일단 absolute를 주고 좌/우 값을 지정해주면 안에 트리거 클릭 영역이 줄어드는 것 같습니다 ㅠㅠ 흠.. 일단 background 넣었을 때 영역은 잘 잡히는데 트리거 영역을 파악하기 어렵네요 ㅠㅠ absolute 사용 안하고 레이아웃 잡아서 수정했습니다.

드롭다운 컴포넌트 테스트 진행했는데 shadcn UI를 사용하다보니 비활성화 상태를 aria-disabled 속성으로 체크하고 있는 것 같아 테스트에서 드롭다운 안에 메뉴 버튼 클릭 비활성화를 판단하기에는 어렵고 aria-disabled 속성이 true인지로 체크하여 테스트 진행했습니다.

여행 정보 드롭다운 메뉴 버튼 중앙이 안눌리는 에러 해결과 불필요한 css 제거
@SSUK-H SSUK-H added 📌fix📌 버그를 수정합니다. 🧪test🧪 labels Feb 21, 2025
@SSUK-H SSUK-H added this to the 1차 MVP QA milestone Feb 21, 2025
@SSUK-H SSUK-H self-assigned this Feb 21, 2025
@github-actions
Copy link

✅ GitHub Actions 실행 결과

  • Build and Test: 성공 🎉
  • Lint: 성공 🎉

Copy link
Contributor

@kwonsuhyuk kwonsuhyuk left a comment

Choose a reason for hiding this comment

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

고생하셨습니당

@SSUK-H SSUK-H merged commit 61bbcae into dev Feb 26, 2025
5 checks passed
@SSUK-H SSUK-H deleted the 187/trip-detail-test-refactor branch February 26, 2025 01:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📌fix📌 버그를 수정합니다. 🧪test🧪

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants