[#179] iOS 17의 TodoListView의 searchable이 포커싱 될 때 툴바가 보이는 현상을 해결한다#180
[#179] iOS 17의 TodoListView의 searchable이 포커싱 될 때 툴바가 보이는 현상을 해결한다#180
Conversation
opficdev
commented
Mar 4, 2026
opficdev
commented
- closed iOS 17의 TodoListView의 searchable이 포커싱 될 때 툴바가 보이는 현상을 해결한다 #179
Summary of ChangesHello, I'm Gemini Code Assist1! 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! 이 Pull Request는 iOS 17에서 Highlights
Changelog
Activity
Using Gemini Code AssistThe 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
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 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
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
이 PR은 iOS 17에서 searchable 사용 시 툴바가 계속 보이는 버그를 수정하는 것을 목표로 하고 있습니다. iOS 18 이상과 미만 버전에 대해 분기 처리를 하여 문제를 해결한 점이 인상적입니다. 코드 구조를 개선하고 불필요한 지역화 문자열을 제거하는 등 좋은 변경 사항들이 포함되어 있습니다. 다만, 리뷰 과정에서 몇 가지 잠재적인 문제점을 발견했습니다. iOS 버전 호환성과 관련된 오타 및 로직 오류가 있어 수정이 필요해 보입니다. 또한, 검색 기능의 사용자 경험이 의도치 않게 변경되었을 가능성이 있어 확인이 필요합니다. 자세한 내용은 각 코드 리뷰 댓글을 참고해주세요.
| if #available(iOS 26.0, *) { | ||
| ToolbarSpacer(.fixed, placement: .topBarTrailing) | ||
| } |
There was a problem hiding this comment.
if #available(iOS 26.0, *) 구문은 오타인 것 같습니다. ToolbarSpacer는 iOS 16.0부터 사용 가능하므로 iOS 16.0으로 수정해야 합니다. 현재 코드는 의도한 대로 동작하지 않을 수 있습니다.
if #available(iOS 16.0, *) {
ToolbarSpacer(.fixed, placement: .topBarTrailing)
}| .searchable( | ||
| text: Binding( | ||
| get: { viewModel.state.searchQuery }, | ||
| set: { viewModel.send(.setSearchQuery($0)) } | ||
| ), | ||
| isPresented: Binding( | ||
| get: { viewModel.state.isSearching }, | ||
| set: { viewModel.send(.setSearching($0)) } | ||
| ), | ||
| placement: .navigationBarDrawer(displayMode: .always), | ||
| prompt: "검색" | ||
| ) |
There was a problem hiding this comment.
searchable 수정자에서 isPresented와 placement 파라미터를 함께 사용하는 오버로드는 iOS 17.0부터 사용 가능합니다. #available(iOS 17.0, *) 확인 로직이 제거되어, 앱의 최소 지원 버전이 iOS 17.0 미만일 경우 컴파일 오류가 발생할 수 있습니다. 다른 파일에서 iOS 16.0을 지원하는 것으로 보이는 코드가 있어, 이 부분을 다시 확인하고 버전 분기 처리를 추가하는 것을 권장합니다.
| if #available(iOS 18, *) { | ||
| ToolbarItem(placement: .topBarTrailing) { | ||
| Button { | ||
| viewModel.send(.setIsSearching(true)) | ||
| } label: { | ||
| Image(systemName: "magnifyingglass") | ||
| } | ||
| } | ||
| } |
There was a problem hiding this comment.
iOS 18 미만 버전에서는 검색 버튼(돋보기 아이콘)이 사라지고, 목록을 아래로 당겨야만 검색창이 나타나도록 변경되었습니다. 이는 기존에 버튼을 통해 검색을 시작하던 방식과 달라진 사용자 경험을 제공합니다. iOS 17의 툴바 버그를 해결하기 위한 의도적인 디자인 변경인지, 아니면 수정 과정에서 발생한 부수적인 효과인지 확인이 필요해 보입니다. 만약 의도하지 않은 변경이라면, iOS 17에서도 검색 버튼을 유지하면서 버그를 해결할 수 있는 다른 방법을 고려해볼 수 있습니다.
1 participant
