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

Popover 의 flag 속성을 추가했다 #35

Merged
merged 2 commits into from
Sep 23, 2023
Merged

Conversation

healtheloper
Copy link
Member

📌 PR 설명

현재 Popover 사용법은

  1. opened prop 을 넘겨주어 opened 의 제어권을 바깥에서 사용하거나
  2. children 이 trigger 가 되어 opened 제어권을 Popover 가 갖거나(currentOpened)

opened 가 변함에 따라 useEffect 에서 currentOpened 를 바꿔주고 있으나 opened 값이 있을 시 Transition 에서 사용되는 값은 opened 이기 때문에 currentOpened 가 바뀌어도 큰 의미가 없었습니다.

opened 대신, flag 값을 사용해서 의도한바를 동작하도록 변경했습니다. 자식이 제어권을 가지고 있더라도 flag 가 변경되면 currentOpened 를 변경시키고, Transition 에서는 opened 값이 없을 시 currentOpened 값을 사용하기 때문에 의도한 바를 동작하게 할 수 있게 됩니다.

@healtheloper healtheloper added the 개선 기능 혹은 비기능적인 요소를 개선합니다. label Sep 22, 2023
@healtheloper healtheloper self-assigned this Sep 22, 2023
@kciter kciter merged commit ffbffb4 into main Sep 23, 2023
2 of 3 checks passed
@kciter kciter deleted the park/popover-open-flag branch September 23, 2023 15:14
@kciter
Copy link
Member

kciter commented Sep 23, 2023

published on 0.9.9

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.

2 participants