Skip to content

[#360] 홈뷰의 웹페이지 섹션의 이미지 크기를 수정한다#361

Merged
opficdev merged 1 commit intodevelopfrom
#360-HomeView-Section-Web
Apr 5, 2026
Merged

[#360] 홈뷰의 웹페이지 섹션의 이미지 크기를 수정한다#361
opficdev merged 1 commit intodevelopfrom
#360-HomeView-Section-Web

Conversation

@opficdev
Copy link
Copy Markdown
Owner

@opficdev opficdev commented Apr 5, 2026

@opficdev opficdev self-assigned this Apr 5, 2026
@opficdev opficdev added the qa TestFlight에 배포 label Apr 5, 2026
Copy link
Copy Markdown
Contributor

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

Choose a reason for hiding this comment

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

Code Review

이 풀 리퀘스트는 WebItemRow의 썸네일 크기 계산 방식을 변경합니다. 화면 너비에 비례한 크기 설정은 iPad의 Split View와 같은 환경에서 UI 일관성을 저해할 수 있으므로, 고정된 크기를 사용하거나 매직 넘버를 상수로 정의하는 개선이 필요합니다.

HStack {
thumbnail
.frame(width: sceneWidth / 10, height: sceneWidth / 10)
.frame(width: sceneWidth * 0.08, height: sceneWidth * 0.08)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

화면 너비(sceneWidth)에 비례하여 이미지 크기를 설정하면 기기 해상도에 따라 UI 일관성이 깨질 수 있습니다. 특히 현재 sceneWidth 구현은 기기의 전체 너비를 반환하므로, iPad의 Split View 등 멀티태스킹 환경에서 이미지가 의도보다 크게 표시될 수 있습니다. 가급적 고정된 크기(Point)를 사용하거나, 0.08과 같은 매직 넘버를 의미 있는 상수로 정의하여 관리하는 것을 권장합니다.

@opficdev opficdev merged commit 471dbf0 into develop Apr 5, 2026
1 check passed
@opficdev opficdev deleted the #360-HomeView-Section-Web branch April 5, 2026 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

qa TestFlight에 배포

Projects

None yet

Development

Successfully merging this pull request may close these issues.

홈뷰의 웹페이지 섹션의 이미지 크기를 수정한다

1 participant