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

SearchView UI 구현 및 데이터 바인딩 #82

Merged
merged 9 commits into from
Mar 5, 2024
Merged

Conversation

eung7
Copy link
Contributor

@eung7 eung7 commented Mar 5, 2024

Screenshots 📸

Simulator.Screen.Recording.-.iPhone.15.Pro.-.2024-03-05.at.10.57.28.mp4



고민, 과정, 근거 💬

  • SearchView UI 구현과 ViewModelView간의 데이터 바인딩을 구현했습니다.
  • 아무래도 해당 View에는 지금 디자인으로 봐선 무한 스크롤은 지원이 안 되지 않을까 싶습니다.
    • 일단 SearchView는 다중 섹션이라서 스크롤을 하고 데이터가 추가되면, 다시 알맞는 섹션에 들어가야 하는데.. 거기서 생기는 문제가 우려되기 때문입니다.. 🥲
  • 다음 PR에서는 SearchView관련 디테일링 작업을 할 예정입니다. 위의 내용도 수정 예정입니다.




@eung7 eung7 added the 🔍 Search Search View label Mar 5, 2024
@eung7 eung7 added this to the v2.0.0 milestone Mar 5, 2024
@eung7 eung7 requested a review from a team March 5, 2024 02:04
@eung7 eung7 self-assigned this Mar 5, 2024
@eung7 eung7 linked an issue Mar 5, 2024 that may be closed by this pull request
- padding(.zero) 제거
- 주석 처리 제거
@eung7 eung7 requested a review from WhiteHyun March 5, 2024 04:54
@eung7 eung7 merged commit 37665c6 into main Mar 5, 2024
2 checks passed
@eung7 eung7 deleted the feature/search/81 branch March 5, 2024 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔍 Search Search View
Projects
Development

Successfully merging this pull request may close these issues.

SearchView UI 구현 및 ViewModel ↔️ View 바인딩
2 participants