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] #16 - 피드뷰 레이아웃 구현 #25

Merged
merged 9 commits into from
Jan 12, 2022

Conversation

yangsubinn
Copy link
Member

🌴 PR 요약

🌱 작업한 브랜치

🌱 작업한 내용

  • FeedHeaderVIew 레이아웃
  • FeedCVC 레이아웃
  • FeedVC 레이아웃
  • 더미데이터 추가 + 무한스크롤 구현
  • UIView+Extension 추가

📌 참고 사항

이게 최선의 코드인지는 잘 모르겠습니다.. 그치만 급한 것 같고.. 정상적으로 작동하며 또 다른 일들이 기다리고 있기 때문에 일단 올립니다!
무한스크롤 구현하는 방식을 전에 연습했던 것과 현규선배가 알려준 방식 두개 다 해봤는데,
현규선배가 알려준 방식에서 isInfiniteScroll 요 친구를 서버 통신할때 true 처리는 것 같은데 확실히 모르겠어서 일단 네.. 둘 다 살려뒀습니다.

📸 스크린샷

기능 스크린샷
피드뷰 ezgif com-gif-maker (29)

📮 관련 이슈

@yangsubinn yangsubinn added 🦹t없e맑은水빈 우리 선배 이젠 타이니하지 않아 🚨review_required🚨 긴급! 빠르게 확인해야할 사항 P1 / Priority High labels Jan 12, 2022
@yangsubinn yangsubinn self-assigned this Jan 12, 2022
Copy link
Member

@hyun99999 hyun99999 left a comment

Choose a reason for hiding this comment

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

서버 붙일때 scrollViewDidScroll 사용해서 무한스크롤 구현하면 될거 같아요!

@yangsubinn yangsubinn merged commit 70f60ac into TeamSparker:develop Jan 12, 2022
@yangsubinn yangsubinn deleted the feature/#16 branch January 27, 2022 06:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 / Priority High 🚨review_required🚨 긴급! 빠르게 확인해야할 사항 🦹t없e맑은水빈 우리 선배 이젠 타이니하지 않아
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] 피드뷰 레이아웃
2 participants