Conversation
There was a problem hiding this comment.
Code Review
This pull request refactors the navigation architecture to support NavigationSplitView for multi-column layouts on larger screens, utilizing a new generic NavigationRouter and tab-specific route enums. It also updates SystemTodoCategoryItem to use UIColor for better system integration and adds localized strings for selection prompts. Feedback identifies that view models in MainView are incorrectly instantiated in computed properties, leading to state loss, and notes redundant background modifiers that should be removed for better code clarity.
There was a problem hiding this comment.
Code Review
이번 PR은 iPad 및 큰 화면 대응을 위해 NavigationSplitView를 도입하고 NavigationRouter를 제네릭하게 개선하여 내비게이션 구조를 리팩토링했습니다. MainView에서 레이아웃에 따라 사이드바와 탭 바를 전환하는 로직이 추가되었으며 각 화면의 경로 처리를 위한 Route 열거형이 도입되었습니다. 피드백으로는 MainView의 연산 프로퍼티 내에서 ViewModel이 반복 생성되어 상태가 초기화되는 문제를 방지하기 위해 @State로 관리할 것과 PushNotificationListView에서 시각적 피드백과 접근성을 위해 onTapGesture 대신 Button을 사용할 것이 제안되었습니다.
🔗 연관된 이슈
📝 작업 내용
📌 요약
🔍 상세
📸 영상 / 이미지 (Optional)