[#359] 하단 탭바 iOS 26 적용 #369
Conversation
Summary of ChangesHello @Hrepay, 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! 이 PR은 기존에 수동으로 구현되었던 커스텀 탭바 시스템을 iOS의 표준 Highlights
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은 기존의 커스텀 탭바를 UITabBarController로 교체하는 중요한 리팩토링을 수행했습니다. 전반적으로 코드 구조가 개선되었고, iOS 표준 컴포넌트를 활용하여 안정성과 유지보수성이 향상되었습니다. 몇 가지 추가적인 개선 사항을 제안합니다. SnapKit import 누락으로 인한 컴파일 오류 수정이 필요하며, setupViewControllers 메서드의 중복 코드를 줄이고, 탭 인덱스에 매직 넘버 대신 enum을 사용하여 코드의 명확성을 높이는 것을 권장합니다. 또한, UIImage extension을 별도 파일로 분리하여 관리하는 것이 좋겠습니다.
EATSSU/App/Sources/Presentation/TabBar/CustomTabBarContainerController.swift
Show resolved
Hide resolved
EATSSU/App/Sources/Presentation/TabBar/CustomTabBarContainerController.swift
Show resolved
Hide resolved
EATSSU/App/Sources/Presentation/TabBar/CustomTabBarContainerController.swift
Show resolved
Hide resolved
| extension UIImage { | ||
| func resized(to size: CGSize) -> UIImage { | ||
| let renderer = UIGraphicsImageRenderer(size: size) | ||
| return renderer.image { _ in | ||
| self.draw(in: CGRect(origin: .zero, size: size)) | ||
| } | ||
|
|
||
| self.tabBarView.alpha = shouldHide ? 0 : 1 | ||
| self.tabBarView.isHidden = shouldHide | ||
| self.view.layoutIfNeeded() | ||
| } | ||
| } |
There was a problem hiding this comment.
UIImage에 대한 확장은 여러 곳에서 재사용될 가능성이 높습니다. Utility/Extension과 같은 공용 폴더에 UIImage+Resize.swift와 같은 별도 파일로 분리하여 관리하는 것이 프로젝트 전체의 코드 구조와 유지보수 측면에서 더 바람직합니다.
There was a problem hiding this comment.
이건 일단 이 파일에서만 사용하고 있기 때문에 유지하겠습니다.
#️⃣ 관련 이슈
Resolved #359
💡작업 내용
시스템 UITabBarController 도입 및 커스텀 탭바 제거
CustomTabBarView) 구현을 제거하고, 표준UITabBarController로 대체했습니다.CustomTabBarContainerController로 통합했습니다.탭바 디자인 및 동작 개선
로그인 권한 제어(Gating) 및 네비게이션 로직 수정
UITabBarControllerDelegate에서 일괄 처리하도록 중앙화했습니다.유틸리티 개선
BaseViewController초기화 시hidesBottomBarWhenPushed설정을 호출하도록 추가하여, 각 뷰 컨트롤러에서 탭바 가시성을 제어할 수 있도록 개선했습니다.💬리뷰 요구사항(선택)