Skip to content

Conversation

@nxnaxx
Copy link
Contributor

@nxnaxx nxnaxx commented Oct 20, 2025

Related issue

Closes #981

Overview

Detail 컴포넌트와 Cluster Graph Tooltip에서 커밋 메시지가 올바르게 표시되도록 수정했습니다.

Work list

1. Detail - 커밋 메시지 분리 및 표시 개선

문제점

  • PR과 연관된 커밋들이 개별 커밋 메시지가 아닌, 동일한 PR 제목/본문으로 표시됨
  • 커밋 본문(body)이 중첩되어 다른 커밋 메시지에 가려지는 현상
overlapping-commit-msg

개선 내용

  • merge 커밋은 title만 표시
  • 개별 커밋은 해당 커밋의 title, body 표시
  • CommitItem hover 시 높이를 확장하여 body 노출 -> hover index 변경 시 cell height 재계산

2. Cluster Graph Tooltip - 메시지 본문 줄바꿈 오류 수정

문제점

  • PR 본문 전체를 렌더링하여 가독성 저하
  • 커밋 body 줄바꿈이 정상적으로 적용되지 않음
cluster-graph-tooltip-issue

개선 내용

  • 1과 동일하게 merge 커밋은 title만 표시
  • 커밋 body에 줄바꿈 처리가 올바르게 적용되도록 수정

Result

fix-commit-msg-display

- Display only title for PR merge commits
- Move commit message splitting logic into CommitItem
- Expand commit body on CommitItem hover
- Adjust scrollToRow to ensure proper spacing when expanding the last row
@nxnaxx nxnaxx self-assigned this Oct 20, 2025
@nxnaxx nxnaxx requested review from a team as code owners October 20, 2025 13:25
Copy link
Contributor

@SingTheCode SingTheCode left a comment

Choose a reason for hiding this comment

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

LGTMMMM!!! 고생하셨습니다ㅎㅎ 훨씬 사용성이 좋아졌네요ㅎㅎ

@yuminnnnni yuminnnnni self-requested a review October 28, 2025 06:02
Copy link
Member

@yuminnnnni yuminnnnni left a comment

Choose a reason for hiding this comment

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

이런 이슈가 있었군요!! 너무 좋은 것 같습니다 👍👍

@nxnaxx nxnaxx merged commit afc872c into githru:main Oct 28, 2025
2 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.

[fix]: Detail, Cluster Graph Tooltip 오류 수정

3 participants