-
Notifications
You must be signed in to change notification settings - Fork 1
[FEAT] 홈 - 장소 없는 경우 UI(대신 여기는 어때요, 다음엔 추천해드릴게요), 서비스불가지역 UI 구현 (#156) #157
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
Conversation
cirtuare
left a comment
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.
체고다 고생했어요 ~~
| override func setLayout() { | ||
| super.setLayout() | ||
|
|
||
| let edge = ScreenUtils.widthRatio * 16 |
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.
🐿️
이거 제가 저번에 SheetUtils에 만들어 뒀습니당 ! 참고해주세요
| $0.height.equalToSuperview() | ||
| $0.leading.equalToSuperview() | ||
| $0.centerY.equalToSuperview() |
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.
🐿️
한 번에 해주면 더 깔꼼할 것 같네용
🐿️ Pull Requests
🪵 작업 브랜치
🥔 작업 내용
메모리 절약을 위해 컬렉션뷰를 하나만 사용하고,
데이터에 따라 Layout과 Cell, Header를 갈아끼우는 방식으로 구현했습니다.
코드 중복을 최소화하기위해 SpotListCellConfigurable 프로토콜을 구현했습니다.
🚨 참고 사항
.bike인 상태에서 tag가 있을 때 아래처럼 레이아웃이 애매한 문제가 있는데, 이건 디자인측에 문의드린 상태입니다.

-> 수정 완료
TempErrorView 사용 뷰는 다른 브랜치에서 수정 후 삭제하겠습니다
📸 스크린샷
💥 To be sure
🌰 Resolve issue