-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feat [#2] TabBar UI 구현 #4
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다. tabBar item을 state Property로 구현한 게 신기하네요~ 스유의 세계란..
근데 브랜치명 컨벤션 지켜주세요!!
@ViewBuilder | ||
var view: some View { | ||
switch self { | ||
case .challengeView: ChallengeView() | ||
case .homeView: HomeView() | ||
case .myPageView: MyPageView() | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 이런 게 있군요 배워갑니다~
selectedTab.view | ||
CustomTabView(selectedTab: $selectedTab) | ||
} | ||
.ignoresSafeArea(.keyboard) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
❓ 탭바 뷰에서는 키보드가 올라올 일이 없을 것 같은데 따로 지정해두신 이유가 궁금합니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
만약 tabBar가 있는 homeView, challengeView, myPageView에서 키보드가 올라오면
탭바가 딸려오는걸 방지하기 위하여 넣었습니다!
사실 지금은 키보드가 올라올 기능이 없어서
불필요할 것 같기도 합니다!!
👾 작업 내용
🚀 PR Point
TabButton 관련
경우들이 많이 보였습니다. 이 방식에 대한 의견이 궁금합니다!
CustomTabBarView
따라서 따로 비활성화된 색 이미지 파일을 다운 받아 이미지 이름에 "Unselected"를 덧 붙여
버튼 클릭 시 이름에 따라 이미지가 교체되도록 구현하였습니다.
-> 더 좋은 방식이 있을 까요?
contentView파일이 불필요하다고 생각합니다!
📸 스크린샷
✅ CheckList
🔗 Issue
Resolved #2