Skip to content

[Feat, Refactor] Gnb: 드롭다운 메뉴 'MemberListMenu' 추가 / hooks: 공통 훅 useClosePopup 추가#105

Merged
MINJI121 merged 7 commits intodevfrom
feature/Gnb
Mar 30, 2025
Merged

[Feat, Refactor] Gnb: 드롭다운 메뉴 'MemberListMenu' 추가 / hooks: 공통 훅 useClosePopup 추가#105
MINJI121 merged 7 commits intodevfrom
feature/Gnb

Conversation

@MINJI121
Copy link
Copy Markdown
Contributor

@MINJI121 MINJI121 commented Mar 29, 2025

변경 사항

  1. MemberListMenu 컴포넌트 추가
    대시보드 인원을 보여주는 드롭다운 메뉴 컴포넌트 추가했습니다.

  2. useClosePopup 공통 컴포넌트 추가
    창 바깥 클릭 시, esc 키다운 시 닫히는 훅 생성,
    공통 컴포넌트 사용 목적으로 src/hooks에 추가했습니다.

사용 방법:
useClosePopup(ref, () => setIsOpen(false));
  1. UserMenu 드롭다운 메뉴에 '내 대시보드' 이동 기능 추가

테스트 유무

화면 캡처 2025-03-30 031053
화면 캡처 2025-03-30 031029
테스트 완료

@MINJI121 MINJI121 added Feat Good for newcomers Refactor Extra attention is needed labels Mar 29, 2025
@MINJI121 MINJI121 self-assigned this Mar 29, 2025
@MINJI121 MINJI121 changed the title [Feat, Refactor] Gnb: 유저 리스트 드롭다운 메뉴 추가 / hooks: 공통 컴포넌트 useOutsideClick 추가 [Feat, Refactor] Gnb: 멤버 리스트 드롭다운 메뉴 추가 / hooks: 공통 훅 useClosePopup 추가 Mar 29, 2025
@MINJI121 MINJI121 changed the title [Feat, Refactor] Gnb: 멤버 리스트 드롭다운 메뉴 추가 / hooks: 공통 훅 useClosePopup 추가 [Feat, Refactor] Gnb: 드롭다운 메뉴 'MemberListMenu' 추가 / hooks: 공통 훅 useClosePopup 추가 Mar 29, 2025
Copy link
Copy Markdown
Contributor

@hhjin1 hhjin1 left a comment

Choose a reason for hiding this comment

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

확인했습니다~!

Copy link
Copy Markdown
Contributor

@gyoyeon-kim gyoyeon-kim left a comment

Choose a reason for hiding this comment

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

확인했습니다!

@MINJI121 MINJI121 merged commit 7f5597e into dev Mar 30, 2025
@MINJI121 MINJI121 deleted the feature/Gnb branch March 31, 2025 04:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feat Good for newcomers Refactor Extra attention is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants