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

[노철] - TooltipButton 구현 #489

Merged
merged 10 commits into from
Mar 13, 2024
Merged

[노철] - TooltipButton 구현 #489

merged 10 commits into from
Mar 13, 2024

Conversation

qkdl60
Copy link
Contributor

@qkdl60 qkdl60 commented Mar 9, 2024

📌 이슈 번호

🚀 구현 내용

올해도-아좌좌-Chrome-2024-03-09-17-49-44

컴포넌트 설명을 주석으로 넣었습니다.

📘 참고 사항

plan 페이지 적용을 했지만 디자인은 변경될 것 같아서 기존 디자인 그래도 사용했습니다.

@qkdl60 qkdl60 added the 🚨긴급 긴급한 확인 필요 label Mar 9, 2024
@qkdl60 qkdl60 requested review from MinwooP and suehdn March 9, 2024 08:56
@qkdl60 qkdl60 self-assigned this Mar 9, 2024
Copy link
Contributor

@suehdn suehdn left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 노철님!!
툴팁 너무 예쁘게 동작하는 것 같네요 😀

isOpen: false, // 툴팁 열림 여부
handleSetIsOpen: () => {}, //툴팁 상태 변경 함수
handleCloseTooltip: () => {}, //툴팁 닫기 함수
optionsPosition: 'top' as Position, //Options컴포넌트 위치(열림 방향을 위해서 필요)
Copy link
Contributor

Choose a reason for hiding this comment

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

top일때는 위로 카카오톡, 링크 복사하기 이렇게 두 개가 보이는 것이고
bottom일 때는 툴팁이 닫히는 방식으로 작동하는 건가요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

컴포넌트 넣어주는 순서에 따라서 팝업이 열리는 위치를 정하도록 해주었습니다
Options의 위치가 top일때는 카카오톡, 링크 복사하기가 위에 나오고, bottom 일때는 아래에 나오도록 설정한겁니다.

<TooltipButton.Main>
  <TooltipButton.Trigger />
  <TooltipButton.Options /> 
</TooltipButton.Main
//Options 컴포넌트가 아래에 있어서 팝업시 버튼 목록이 아래로 펼쳐집니다. 

<TooltipButton.Main>
<TooltipButton.Options/>
<TooltipButton.Trigger/>
</TooltipButton.Main>
// Options 컴포넌트가 위에 있어서 팝업시 버튼 목록이 위로 펼쳐집니다.

Copy link
Contributor

Choose a reason for hiding this comment

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

작성 위치에 따라 변경되는 부분 너무 좋은 것 같은데 처음 보는 사람은 이해하기가 좀 어려운것 같습니다..!
props로 받아와 변경해주는게 좀 더 사용하기 편할것 같아요 😄

@suehdn suehdn self-requested a review March 11, 2024 06:34
Copy link
Contributor

@MinwooP MinwooP left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!! 툴팁버튼 너무 이쁘네요 👍🏻 컴포넌트 설계하면서 많은 고민하신게 느껴지네요!ㅎㅎㅎ

@qkdl60 qkdl60 merged commit 9919f2f into dev Mar 13, 2024
1 check passed
@qkdl60 qkdl60 deleted the feat/#488/tootip-button branch March 13, 2024 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚨긴급 긴급한 확인 필요
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[노철] - tooltip button 구현
3 participants