Skip to content
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] 바텀시트 내 검색기능 구현 #465

Merged
merged 4 commits into from
Sep 19, 2022

Conversation

jane1choi
Copy link
Member

@jane1choi jane1choi commented Sep 15, 2022

🍎 관련 이슈

closed #460

🍎 변경 사항 및 이유

바텀시트 내 검색기능을 구현했습니다.

🍎 PR Point

  • UITableViewDiffableDataSource를 이용해 검색어가 바뀔 때마다 검색결과에 반영될 수 있도록 검색 기능을 구현했습니다.
    UITableViewDiffableDataSource 공식문서
  • UITableViewDiffableDataSource 요 친구가 UITableViewDataSource를 대체하기 때문에 기존 UITableViewDataSource extension을 다 삭제해주었습니다 참고해주세요!
  • 검색어가 없다면 기존 리스트 전체가 나오도록 처리해주었습니다!
  • 궁금한 점 있다면 질문환영합니다~!

📸 ScreenShot

Simulator.Screen.Recording.-.iPhone.11.-.2022-09-15.at.20.49.53.mp4

@jane1choi jane1choi added 📱 View 뷰 개발(디자이닝)시 사용합니다. 🍎 Feature 새로운 기능 개발시 사용합니다. 💜 은주 💜 은주가 연 pr, issue에 사용 ✏️ 과방탭 과방탭 개발시 사용합니다. labels Sep 15, 2022
@jane1choi jane1choi added this to In progress in ✨ NadoSunbae ✨ via automation Sep 15, 2022
@jane1choi jane1choi added this to In progress in NadoSunbae-iOS for release✨ via automation Sep 15, 2022
@jane1choi jane1choi self-assigned this Sep 15, 2022
Copy link
Member

@dev-jungbin dev-jungbin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했어요~!! 새로운 거 봐서 신기하다... 그리고 data 바뀔 때 애니메이션도 예쁘당...

Comment on lines +158 to +163
self.dataSource = UITableViewDiffableDataSource<Section, MajorInfoModel>(tableView: self.majorTV) { (tableView, indexPath, majorName) -> UITableViewCell? in
guard let cell = tableView.dequeueReusableCell(withIdentifier: MajorTVC.className, for: indexPath) as? MajorTVC else { preconditionFailure() }

cell.cellType = self.cellType
cell.setData(majorName: majorName)
return cell
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

오 기존 DataSource 대신 이렇게 사용하는구나`!!!

Copy link
Member

@hwangJi-dev hwangJi-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

textField가 변화할때마다 snapshot을 이용해서 데이터소스를 변화시켜주는구나 새로운 코드 알아갑니당~~ 고생했어!

@jane1choi jane1choi merged commit bd9981e into develop Sep 19, 2022
✨ NadoSunbae ✨ automation moved this from In progress to Done Sep 19, 2022
NadoSunbae-iOS for release✨ automation moved this from In progress to Done Sep 19, 2022
@jane1choi jane1choi deleted the feature/#460-Func-BottomSheet-Search branch September 19, 2022 05:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✏️ 과방탭 과방탭 개발시 사용합니다. 🍎 Feature 새로운 기능 개발시 사용합니다. 💜 은주 💜 은주가 연 pr, issue에 사용 📱 View 뷰 개발(디자이닝)시 사용합니다.
Development

Successfully merging this pull request may close these issues.

✨ [FEAT] 바텀시트 내 검색기능 구현
3 participants