[#312] TodayView에서 Todo를 직접 완료할 수 있는 UI를 구성한다#331
Merged
Conversation
Owner
opficdev
commented
Mar 29, 2026
opficdev
commented
- closed TodayView에서 Todo를 직접 완료할 수 있는 UI를 구성한다 #312
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Code Review
이번 풀 리퀘스트는 TodayViewModel의 SectionBuckets를 SectionStore로 이름을 변경하고, TodayView에서 할 일 완료를 위한 스와이프 액션을 개선하는 내용을 담고 있습니다. 피드백으로는 Store 프로토콜과의 혼동을 피하기 위한 구조체 이름 변경 제안과, 의도치 않은 완료 처리를 방지하기 위해 swipeActions에 allowsFullSwipe: false 옵션을 추가하라는 의견이 있었습니다.
| } | ||
|
|
||
| struct SectionBuckets { | ||
| struct SectionStore { |
Contributor
There was a problem hiding this comment.
SectionStore라는 이름은 ViewModel이 Store 프로토콜을 따르고 있어 혼동을 줄 수 있습니다. 이 구조체는 단순히 데이터를 그룹핑하는 역할을 하므로 GroupedSections 또는 SectionCollection 같은 더 명확한 이름으로 변경하는 것을 고려해보세요. 이는 코드의 의도를 더 명확하게 전달하는 데 도움이 됩니다.
Suggested change
| struct SectionStore { | |
| struct GroupedSections { |
DevLog/UI/Today/TodayView.swift
Outdated
| .tint(.orange) | ||
|
|
||
| } | ||
| .swipeActions(edge: .trailing) { |
Contributor
There was a problem hiding this comment.
swipeActions의 allowsFullSwipe 파라미터가 설정되지 않아 기본값인 true가 적용됩니다. 이 경우 사용자가 실수로 전체 스와이프하여 Todo를 완료 처리할 수 있습니다. togglePinned 액션처럼 allowsFullSwipe: false를 명시적으로 설정하여 사용자의 실수를 방지하는 것을 고려해보세요.
Suggested change
| .swipeActions(edge: .trailing) { | |
| .swipeActions(edge: .trailing, allowsFullSwipe: false) { |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.