Skip to content

Conversation

@yurim830
Copy link
Collaborator

@yurim830 yurim830 commented May 30, 2025

🐿️ Pull Requests

🪵 작업 브랜치

🥔 작업 내용

  • 메모리 절약을 위해 컬렉션뷰를 하나만 사용하고,
    데이터에 따라 Layout과 Cell, Header를 갈아끼우는 방식으로 구현했습니다.

    • transportMode == .walk -> SpotListCollectionViewFlowLayout, SpotListCell, SpotListHeader
    • transportMode == .bike -> 일반 FlowLayout, NoMatchingSpotCell, NoMatchingHeader
    • transportMode == nil -> 일반 FlowLayout, NoMatchingHeader
  • 코드 중복을 최소화하기위해 SpotListCellConfigurable 프로토콜을 구현했습니다.

🚨 참고 사항

  • .bike인 상태에서 tag가 있을 때 아래처럼 레이아웃이 애매한 문제가 있는데, 이건 디자인측에 문의드린 상태입니다.

    -> 수정 완료

  • TempErrorView 사용 뷰는 다른 브랜치에서 수정 후 삭제하겠습니다

📸 스크린샷

대신 여기는 어때요 다음엔 추천해드릴게요 서비스불가지역 이미지없는식당(.bike) UI 변경사항 반영

💥 To be sure

  • 모든 뷰가 잘 실행되는지 다시 한 번 체크해주세요 !

🌰 Resolve issue

@yurim830 yurim830 requested a review from cirtuare May 30, 2025 13:40
@yurim830 yurim830 self-assigned this May 30, 2025
@yurim830 yurim830 added 🌀 feature 새로운 기능 개발 🥑 유림 유림 labels May 30, 2025
@yurim830 yurim830 added this to the Sprint - 3 milestone May 30, 2025
Copy link
Contributor

@cirtuare cirtuare left a 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
Copy link
Contributor

Choose a reason for hiding this comment

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

🐿️
이거 제가 저번에 SheetUtils에 만들어 뒀습니당 ! 참고해주세요

Comment on lines 56 to 58
$0.height.equalToSuperview()
$0.leading.equalToSuperview()
$0.centerY.equalToSuperview()
Copy link
Contributor

Choose a reason for hiding this comment

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

🐿️
한 번에 해주면 더 깔꼼할 것 같네용

@yurim830 yurim830 merged commit 08d49be into develop May 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🌀 feature 새로운 기능 개발 🥑 유림 유림

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEAT] 홈 - 장소 없는 경우 UI(대신 여기는 어때요, 다음엔 추천해드릴게요), 서비스불가지역 UI 구현

3 participants