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

✨ 캐러셀 및 사용자 기본 이미지 추가 #50

Merged
merged 9 commits into from
May 11, 2024

Conversation

BangDori
Copy link
Collaborator

@BangDori BangDori commented May 11, 2024

작업 이유

  • 피드에서 캐러셀(이미지 슬라이드) 추가
  • 사용자 기본 이미지 추가
  • 기타 작업

작업 사항

1️⃣ 캐러셀 기능

  • 캐러셀 테스트를 위해 피드 목 데이터에 이미지 목록 수정하였습니다.
  • 캐러셀 기능을 구현하면서 피드 스타일 코드를 수정하였습니다. (간격은 기존과 동일하게 적용됩니다.)

이미지가 2개 이상인 경우 컨트롤 박스가 표시됩니다.

이미지가 1개인 경우 컨트롤 박스가 표시되지 않습니다.

2️⃣ 사용자 기본 이미지 추가

사용자의 이미지가 없는 경우, 사용자 기본 이미지가 렌더링 되도록 수정하였습니다.

3️⃣ 기타 작업

  • 스크롤 바 설정 아이폰 Layout에서 전역 설정으로 수정하였습니다.
  • 캐러셀 및 이미지의 크기 비율에 맞춰 이미지를 받아오도록 mock 이미지 경로를 수정하였습니다! (https://picsum.photos/200/200 -> https://picsum.photos/320/400)

리뷰어가 중점적으로 확인해야 하는 부분

  • 캐러셀이 정상적으로 적용되었나요?
  • UI에 이상이 없나요? (캐러셀 동작, 간격 등)

발견한 이슈

  • 없음

@BangDori BangDori requested a review from Legitgoons May 11, 2024 15:16
@BangDori BangDori self-assigned this May 11, 2024
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-50.d37mn03xh3qyyz.amplifyapp.com

Copy link
Member

@Legitgoons Legitgoons left a comment

Choose a reason for hiding this comment

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

의도한 대로 문제없이 동작합니다. 병합하셔도 좋습니다.

@BangDori BangDori merged commit 68f618a into main May 11, 2024
2 checks passed
@BangDori BangDori deleted the feature/feed-carousel branch May 11, 2024 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants