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

모달 관련 오류 해결 #427

Merged
merged 10 commits into from
Sep 24, 2023
Merged

모달 관련 오류 해결 #427

merged 10 commits into from
Sep 24, 2023

Conversation

na-reum
Copy link
Contributor

@na-reum na-reum commented Sep 17, 2023

🚩 관련 이슈

📋 작업 내용

  • esc 누르면 왜 모달 안닫겨?
  • 모달 닫았을때 왜 스크롤 안돼?

📌 PR Point

  • 은수의 탐색으로 인하여 headlessUiDialog 컴포넌트는 컴포넌트 상에 Nesting을 해주면 중첩모달의 동작이 정상적으로 이루어짐을 알아냄
  • 그로인하여 useOverlay를 걷어내고 Nesting 하는 구조로 가져가서 중첩모달을 만들었음.

📸 스크린샷

@na-reum na-reum temporarily deployed to development September 17, 2023 13:20 — with GitHub Actions Inactive
@na-reum na-reum changed the title feat: 오류 수정 모달 관련 오류 해결 Sep 17, 2023
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Sep 17, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: d66f2f2
Status: ✅  Deploy successful!
Preview URL: https://f22aae65.sopt-crew-dev-legacy.pages.dev
Branch Preview URL: https://feat--424.sopt-crew-dev-legacy.pages.dev

View logs

@na-reum na-reum temporarily deployed to development September 17, 2023 13:43 — with GitHub Actions Inactive
@na-reum na-reum temporarily deployed to development September 17, 2023 13:55 — with GitHub Actions Inactive
@na-reum na-reum temporarily deployed to development September 18, 2023 02:33 — with GitHub Actions Inactive
<ModalBackground />
<Dialog.Panel>{children}</Dialog.Panel>
</Dialog>
<Show
Copy link
Contributor

Choose a reason for hiding this comment

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

이렇게 fallback 으로 조건부 렌더링하는 거 흥미롭다..!!

@na-reum na-reum temporarily deployed to development September 19, 2023 10:07 — with GitHub Actions Inactive
Copy link
Member

@100Gyeon 100Gyeon left a comment

Choose a reason for hiding this comment

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

좋습니당 👍🏻👍🏻👍🏻

@na-reum na-reum temporarily deployed to development September 23, 2023 08:49 — with GitHub Actions Inactive
@pull-request-size pull-request-size bot added size/L and removed size/M labels Sep 23, 2023
Copy link
Member

@eunsukimme eunsukimme left a comment

Choose a reason for hiding this comment

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

고생했어요~~ 👍

src/hooks/useOverlay/OverlayController.tsx Show resolved Hide resolved
src/components/util/Show.tsx Outdated Show resolved Hide resolved
Copy link
Member

Choose a reason for hiding this comment

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

요 로직도 필요없어 지는 거겠쥐?

Copy link
Member

Choose a reason for hiding this comment

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

요거는 overlay provider랑 headless-ui를 같이 쓸때 스크롤 안되는 이슈를 예방하기 위해 필요한 것이라고 하여 남겨둡시다

@na-reum na-reum temporarily deployed to development September 24, 2023 02:38 — with GitHub Actions Inactive
@na-reum na-reum merged commit fe13016 into develop Sep 24, 2023
2 checks passed
@na-reum na-reum deleted the feat/#424 branch September 24, 2023 03:00
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.

CreateFeedModal 버그 수정
4 participants