Skip to content

[#323] iOS 18이하에서 HomeView의 최근 수정된 섹션의 row의 패딩이 어색한 현상을 해결한다#332

Merged
opficdev merged 1 commit intodevelopfrom
fix/#313-HomeView
Mar 29, 2026
Merged

[#323] iOS 18이하에서 HomeView의 최근 수정된 섹션의 row의 패딩이 어색한 현상을 해결한다#332
opficdev merged 1 commit intodevelopfrom
fix/#313-HomeView

Conversation

@opficdev
Copy link
Copy Markdown
Owner

@opficdev opficdev self-assigned this Mar 29, 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

이 풀 리퀘스트는 HomeView 내 RecentTodoRow에 적용되어 있던 음수 수직 패딩을 제거합니다. 리뷰어는 이로 인해 다른 UI 요소와의 일관성이 부족해질 수 있음을 지적하며, RecentTodoRow 내부에 직접 패딩을 추가하여 일관성을 확보하고 뷰의 책임을 명확히 할 것을 제안했습니다.

ForEach(viewModel.state.recentTodos, id: \.id) { todo in
NavigationLink(value: Path.detail(todo.id)) {
RecentTodoRow(todo: todo, sceneWidth: sceneWidth)
.padding(.vertical, -4)
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

.padding(.vertical, -4)를 제거하여 List의 기본 패딩을 사용하도록 수정하셨습니다. 이 변경으로 인해 RecentTodoRow와 시각적으로 유사한 TodoItemRow(.padding(.vertical, 12) 사용) 간의 UI 일관성이 부족해질 수 있습니다. RecentTodoRow 자체에 수직 패딩을 추가하여 일관성을 확보하고 뷰의 책임을 명확히 하는 것을 고려해 보세요. 예를 들어, RecentTodoRow의 최상위 HStack.padding(.vertical, 8)과 같이 적절한 패딩을 추가할 수 있습니다.

@opficdev opficdev merged commit 91c82cd into develop Mar 29, 2026
1 check passed
@opficdev opficdev deleted the fix/#313-HomeView branch March 29, 2026 09:30
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.

iOS 18 이하에서 최근 수정된 Todo를 보여주는 row의 패딩이 부족한 현상을 해결한다

1 participant