Skip to content

링크카드상세패널 수정 시 툴팁 노출#446

Merged
Seong-Myeong merged 1 commit intomainfrom
feature/#445-fix-linkcardDetailPanel-tooltip
Mar 25, 2026
Merged

링크카드상세패널 수정 시 툴팁 노출#446
Seong-Myeong merged 1 commit intomainfrom
feature/#445-fix-linkcardDetailPanel-tooltip

Conversation

@Seong-Myeong
Copy link
Copy Markdown
Contributor

@Seong-Myeong Seong-Myeong commented Mar 24, 2026

관련 이슈

PR 설명

  • 링크카드상세패널에서 제목과 메모 수정시에 툴팁이 지속적으로 노출되는 부분을 수정하였습니다.
  • 수정 내용
    • Tooltip.tsx에 disabled 추가
    • TitleSection.tsx, MemoSection.tsx 적용 및 요청 멘트 변경
KakaoTalk_Video_2026-03-24-21-20-05-1.mp4

@Seong-Myeong Seong-Myeong self-assigned this Mar 24, 2026
@Seong-Myeong Seong-Myeong linked an issue Mar 24, 2026 that may be closed by this pull request
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 24, 2026

개요

Tooltip 컴포넌트에 disabled prop을 추가하고, disabled 상태일 때 tooltip이 즉시 닫히고 열기 지연 타이머가 중단되도록 수정했습니다. LinkCardDetailPanel의 제목과 메모 입력 섹션에서 편집 중일 때 해당 tooltip을 비활성화하는 로직을 적용했습니다.

관련 PR

  • Tooltip 컴포넌트 구현 #157: Tooltip 컴포넌트의 기본 구현 및 이벤트 핸들러 로직을 제공하며, 이번 PR은 해당 구현을 확장하여 disabled prop과 상태별 이벤트 바인딩 로직을 추가합니다.
🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed 풀 리퀘스트 제목이 변경 사항의 핵심을 명확하게 요약하고 있습니다.
Linked Issues check ✅ Passed Tooltip에 disabled prop을 추가하고 TitleSection, MemoSection에 적용하여 편집 중 툴팁 노출 문제를 해결했습니다.
Out of Scope Changes check ✅ Passed 모든 변경사항이 링크카드 상세 패널 툴팁 노출 문제 해결이라는 이슈 #445의 목표와 직접적으로 관련되어 있습니다.
Description check ✅ Passed PR 설명이 필수 섹션을 모두 포함하고 있으며, 변경 사항과 목표가 명확하게 기술되어 있습니다.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/#445-fix-linkcardDetailPanel-tooltip

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Seong-Myeong Seong-Myeong merged commit 0f0ea04 into main Mar 25, 2026
3 checks 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.

링크상세패널 툴팁 노출

2 participants