Skip to content

fix(desktop): use explicit info button instead of row tap gesture - #9

Merged
seungwonme merged 1 commit into
mainfrom
worktree-youtube-info-button
Jul 4, 2026
Merged

fix(desktop): use explicit info button instead of row tap gesture#9
seungwonme merged 1 commit into
mainfrom
worktree-youtube-info-button

Conversation

@seungwonme

Copy link
Copy Markdown
Owner

문제

채널 행 전체에 건 onTapGesture가 옆의 삭제(휴지통) Button과 같은 컨테이너에 있어 제스처 인식이 씹혀서, 클릭해도 브라우저가 아예 뜨지 않았음(PR #7 후속 리포트, 사용자 확인: "브라우저가 안 뜬다").

수정

행 전체 tap 대신, 이미 확실히 동작하는 삭제 버튼과 동일한 Button 패턴으로 명시적인 정보(info.circle) 버튼을 추가해 openSourcePage를 호출하도록 변경.

테스트

  • swift build --package-path apps/desktop 성공
  • 기존 Python 테스트 214개 통과 (pre-commit hook)

⚠️ GUI 클릭 동작 자체는 원격 환경이라 직접 검증하지 못했습니다. 머지 후 실제 클릭으로 확인 부탁드립니다.

Row-level onTapGesture wasn't firing reliably next to the sibling
delete Button, so clicking a channel never opened a browser.
@seungwonme
seungwonme merged commit 34822ad into main Jul 4, 2026
3 checks passed
@seungwonme
seungwonme deleted the worktree-youtube-info-button branch July 4, 2026 13:36

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request replaces the row-level tap gesture for opening a source page with an explicit info button. The reviewer suggests dynamically updating the button's tooltip message based on the source platform (e.g., YouTube vs. other platforms) to avoid misleading users when non-YouTube sources are displayed.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment on lines +686 to +692
Button {
openSourcePage(source)
} label: {
Image(systemName: "info.circle")
}
.buttonStyle(.borderless)
.help("유튜브 채널 정보 열기")

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

⚠️ 툴팁 메시지 개선 제안

TrackedSource는 유튜브(youtube) 외의 다른 플랫폼도 지원할 수 있으며, openSourcePage(_:) 메서드에서도 source.platform == "youtube"가 아닐 경우 handleOrURL을 통해 해당 플랫폼의 웹페이지를 열도록 분기 처리되어 있습니다.

따라서 툴팁(.help) 메시지를 "유튜브 채널 정보 열기"로 고정하면, 유튜브가 아닌 다른 플랫폼의 소스일 때 사용자에게 오해를 불러일으킬 수 있습니다.

플랫폼에 따라 툴팁 메시지를 동적으로 변경하도록 수정하는 것을 권장합니다.

Suggested change
Button {
openSourcePage(source)
} label: {
Image(systemName: "info.circle")
}
.buttonStyle(.borderless)
.help("유튜브 채널 정보 열기")
Button {
openSourcePage(source)
} label: {
Image(systemName: "info.circle")
}
.buttonStyle(.borderless)
.help(source.platform == "youtube" ? "유튜브 채널 정보 열기" : "소스 페이지 열기")

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant