[#177] PushNotificationListView의 시트에서 툴바가 보이도록 하고, 수정 버튼이 없도록 구현한다

[opficdev]

• closed PushNotificationListView의 시트에서 툴바가 보이도록 하고, 수정 버튼이 없도록 구현한다 #177

[gemini-code-assist]

Summary of Changes

Hello, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 풀 리퀘스트는 TodoDetailView의 재사용성과 프레젠테이션 유연성을 향상시킵니다. 상세 보기 내에서 편집 버튼의 표시 여부를 제어하는 메커니즘을 도입하여, 다양한 컨텍스트에서 해당 버튼의 모양을 사용자 정의할 수 있도록 합니다. 특히, PushNotificationListView에서 TodoDetailView가 시트로 표시될 때, 편집 버튼이 숨겨지고 툴바에 적절한 닫기 버튼이 제공되어 알림을 통해 작업을 확인하는 사용자 경험이 개선되었습니다.

Highlights

• 편집 버튼 표시 제어: TodoDetailViewModel에 showEditButton 매개변수를 추가하여 편집 버튼의 표시 여부를 제어할 수 있도록 했습니다.
• 조건부 편집 버튼 렌더링: TodoDetailView는 이제 이 새로운 뷰 모델 속성에 따라 편집 버튼을 조건부로 표시합니다.
• PushNotificationListView 시트 개선: PushNotificationListView는 TodoDetailView를 NavigationStack 시트로 표시하며, 이 특정 컨텍스트에서는 편집 버튼을 명시적으로 숨기고 툴바에 닫기 버튼을 추가했습니다.

Changelog

• DevLog/Presentation/ViewModel/TodoDetailViewModel.swift
  • showEditButton 속성을 추가하여 편집 버튼의 표시 여부를 제어합니다.
• DevLog/UI/Home/TodoDetailView.swift
  • 뷰 모델의 showEditButton 값에 따라 편집 버튼이 조건부로 표시되도록 수정했습니다.
• DevLog/UI/PushNotification/PushNotificationListView.swift
  • TodoDetailView를 NavigationStack으로 감싸 시트 형태로 표시하도록 변경했습니다.
  • 시트에서 TodoDetailView를 초기화할 때 편집 버튼을 숨기도록 설정했습니다.
  • 시트의 툴바에 닫기 버튼을 추가했습니다.

Activity

• 풀 리퀘스트 설명에 "closed PushNotificationListView의 시트에서 툴바가 보이도록 하고, 수정 버튼이 없도록 구현한다 #177"만 언급되어 있어, 이 풀 리퀘스트 내에서 명시적인 활동(예: 댓글, 검토)은 기록되지 않았습니다.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature	Command	Description
Code Review	/gemini review	Performs a code review for the current pull request in its current state.
Pull Request Summary	/gemini summary	Provides a summary of the current pull request in its current state.
Comment	@gemini-code-assist	Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help	/gemini help	Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

[gemini-code-assist]

Code Review

안녕하세요. 수석 iOS 개발자입니다. Pull Request를 검토했습니다.

PushNotificationListView의 시트에서 TodoDetailView를 보여줄 때, 수정 버튼을 숨기고 툴바를 추가하는 변경 사항을 확인했습니다. TodoDetailViewModel에 showEditButton 플래그를 추가하여 뷰의 재사용성을 높인 점이 좋습니다. 또한 NavigationStack을 사용하여 시트 내에서 툴바가 올바르게 표시되도록 구현한 것도 적절합니다.

다만, TodoDetailView에서 ToolbarSpacer를 사용할 때 #available 체크 버전이 iOS 26.0으로 되어있는 부분을 발견했습니다. 이는 오타로 보이며, 의도한 UI가 표시되지 않는 버그를 유발할 수 있습니다. 해당 부분에 대한 수정을 제안하는 리뷰 코멘트를 남겼습니다.

전반적으로 PR의 목적에 맞게 코드가 잘 작성되었습니다. 제안된 수정 사항을 반영해주시면 감사하겠습니다.