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

[Fix] 쌓임맥락 고려하여 Tooltip 재배치 #314

Merged
merged 2 commits into from
Nov 6, 2024

Conversation

namdaeun
Copy link
Member

@namdaeun namdaeun commented Nov 5, 2024

해당 이슈 번호

closed #313


체크리스트

  • 🔀 PR 제목의 형식을 잘 작성했나요? e.g. [feat] PR을 등록한다.
  • 💯 테스트는 잘 통과했나요?
  • 🏗️ 빌드는 성공했나요?
  • 🧹 불필요한 코드는 제거했나요?
  • ✅ 컨벤션을 지켰나요?
  • 💭 이슈는 등록했나요?
  • 🏷️ 라벨은 등록했나요?
  • 💻 git rebase를 사용했나요?
  • 🙇‍♂️ 리뷰어를 지정했나요?

📌 내가 알게 된 부분


쌓임 맥락이라는 개념에 대해 막연하게 듣기만 했는데 이걸 이제야 개념을 이해한 것 같습니다.

현재 Tooltip은 SideNavBar 컴포넌트 내부에 위치하고 있으며, SideNavBar 구조는 아래와 같습니다:

      <Login>
        <ModalContainer />

        <SideNavBar />

        <div css={layoutStyle}>
          <main css={outletStyle}>
            <Header />
            <Outlet />
          </main>
          <GlobalDrawer />
        </div>
      </Login>

Tooltip이 다른 컴포넌트에 의해 가려지는 문제의 원인은 부모 컴포넌트인 SideNavBar에 쌓임 맥락이 생성되어 있지 않기 때문입니다. 여기서 "쌓임 맥락이 생성된다"는 것은 z-index가 auto가 아닌 값으로 지정되어 있는 것을 의미합니다.

Tooltip 컴포넌트는 SideNavBar의 쌓임 맥락을 따르기 때문에, 단순히 z-index를 부여해도 부모의 쌓임 맥락에 가로막혀 contentBox에 의해 가려지는 문제가 발생한 것입니다.

따라서 SideNavBar의 스타일에 zIndex를 부여함으로써 해결할 수 있었어요.

아티클 참고하시면 좋을 것 같습니다

📌스크린샷 (선택)

2024-11-05.5.07.27.mov

Copy link
Contributor

@Bowoon1216 Bowoon1216 left a comment

Choose a reason for hiding this comment

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

굳굳 짱짱입니다 수고하셨습니다!

@namdaeun namdaeun changed the title [Fix] Tooltip 위로 배치 [Fix] 쌓임맥락 고려하여 Tooltip 재배치 Nov 5, 2024
Copy link
Contributor

@wuzoo wuzoo left a comment

Choose a reason for hiding this comment

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

고생고생 하셧슴다 ~

Copy link
Contributor

@rtttr1 rtttr1 left a comment

Choose a reason for hiding this comment

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

LGTM!!!

Copy link
Contributor

@cindy-chaewon cindy-chaewon left a comment

Choose a reason for hiding this comment

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

레전드 개발자 당니언니!!

@namdaeun namdaeun merged commit 23ed69f into develop Nov 6, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

툴팁 가려지는 이슈 해결
5 participants