🐛 [BugFix] 전역 모달을 close 시 unmount

[yoopark]

Summary

ModalProvider에서 전역 모달을 관리하는 것은 동일.

하지만, 전역 모달의 경우 close 되었을 때 unmount가 되어야 함. (아래 ModalProvider.tsx의 변경점 참고)

background에서 계속 keydown을 listening 하고 있는 버그가 있었음.

Describe your changes

Mac + K 버튼으로 Spotlight를 활성화할 수 있습니다.

하지만 기존 ModalProvider의 방식은 open & close 가 mount & unmount를 불러오지 않습니다.

전역 모달은 App mount 시에 함께 mount 되며, open & close 는 mount & unmount에 영향을 주지 않았습니다.

일례로, Spotlight를 끄고 페이지를 움직이지 않은 채 다시 켜면 예전 검색어가 남아있었습니다.

그것을 나름의 기능이라 생각하고 넘겼었는데, 이번 #199 이슈를 살펴보면서 Spotlight의 ArrowUp, ArrowDown 이 close 상태에도 listening 하고 있는 것이 원인임을 발견하였습니다.

더하여, Spotlight에서 검색한 이후 Spotlight를 끄고 ArrowUp, ArrowDown을 누르면 Spotlight의 Focus가 움직여버리는 버그가 있다는 사실을 알게 되었습니다.

따라서 기존 @jpham005 님과 얘기를 나눈 방식대로, close 시 전역 모달을 unmount 시키는 방식으로 로직을 변경하게 되었습니다.

Issue number and link

• close Command + ↑↓ 스크롤 단축키 바인딩 #199

[jpham005]

dialog 에 isOpen 을 전달하는 이유가 있을까요?

[yoopark]

dialog 에 isOpen 을 전달하는 이유가 있을까요?

전역 모달의 경우에는 항상 true 이겠군요.

Dialog가 isOpen을 prop으로 받기는 해서, isOpen={isSpotlightOpen}을 isOpen으로 수정하겠습니다.