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.

Sign up for GitHub

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

Jump to bottom

Tooltip 의 visible 로직 수정했다 #45

Merged

kciter merged 1 commit into main from park/tooltip

Nov 17, 2023

Member

healtheloper commented Nov 17, 2023

📌 PR 설명

이전 PR 에서는 visible prop 을 통해 Tooltip 의 visible 을 제어하려 했으나 Tooltip 이 가지는 기본 기능(handleMouseUp 등)을 사용하여 currentVisible 값을 컨트롤 하려면 onChangeVisible 함수를 사용해야 하고, 이 함수와 Tooltip 함수를 사용하는 곳에서 visible 관련 setter 함수를 사용하면 상태가 계속 바뀌는 현상이 있어 Popover 와 같이 flag 로 제어하는 것으로 수정하였습니다.

healtheloper requested a review from kciter

November 17, 2023 10:39

healtheloper self-assigned this


          Tooltip 의 visible 로직 수정했다

c5709f1

healtheloper force-pushed the park/tooltip branch from 799932b to c5709f1 Compare

November 17, 2023 10:52

kciter merged commit 892fea4 into main

1 check failed

kciter deleted the park/tooltip branch

November 17, 2023 10:54

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet