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.

Sign up for GitHub

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

Jump to bottom

[FEAT] #2 컨텐츠 세부화면 UI 구성 #26

Merged
merged 7 commits into from Jul 12, 2022
Merged

[FEAT] #2 컨텐츠 세부화면 UI 구성 #26

merged 7 commits into from Jul 12, 2022

Conversation

jayden000106
Copy link
Collaborator

세부 사항

  • 컨텐츠 세부화면 UI 구성
  • 상단 이미지, 함께 시청된 컨텐츠 그리드는 이미지가 아닌 임시로 RoundedRectangle로 구현
  • 텍스트도 Text("리터럴")로 값을 직접 넣어서 추후 수정 예정
  • 출연, 감독 텍스트는 나누고, 더보기는 버튼으로 구현

질문 사항

  • 상단 모서리만 둥글게 구현하려면 다른 쉬운 방법이 있는지?

체크 리스트 (아래 사항들이 전부 체크되어야만 merge)

  • 필요한 test들을 완료하였고 기능이 제대로 실행되는지 확인 하였습니다.
  • 코드 스타일 가이드에 맞추어 코드를 작성 하였습니다.
  • 제가 의도한 파일들과 수정 사항들만 커밋이 된 것을 확인 하였습니다.
  • 본 수정 사항들을 팀원들과 사전에 상의하였고 팀원들 모두 해당 PR에 대하여 알고 있습니다.

@jayden000106
[FEAT] 컨텐츠 세부 화면 뷰 추가
ab8d1a9
@jayden000106
[FEAT] 상단 예고편 또는 스틸이미지를 위한 사각형 구현
905f387 
- 검은 배경, 다크 테마 적용
- 상단 두 모서리만 둥근 사각형 구현
@jayden000106
[FEAT] commitcomplete#2 영화 타이틀 텍스트 구현
cf62049 
- "이상한 나라의 수학자"로 문자열 리터럴을 임시로 넣음(Fix Needed)
@jayden000106
[FEAT] commitcomplete#2 컨텐츠 세부 정보 텍스트 구현
e752f60 
- 더미 데이터를 텍스트에 넣어서 보여줌(Fix Needed)
@jayden000106
[FEAT] commitcomplete#2 하단 세부 텍스트 구현
a3fe011 
- 스크롤 뷰 사용
- Text에 리터럴을 직접 사용(Fix Needed)
- 더보기 버튼 구현
@jayden000106
[FEAT] commitcomplete#2 하단 함께 시청된 콘텐츠가 보이는 GridView 구현
be14718 
- RoundedRectangle을 이용한 크기 설정
@jayden000106 jayden000106 added this to In progress in Netflix CloneCoding Jul 10, 2022
@jayden000106 jayden000106 added the enhancement New feature or request label Jul 10, 2022
@jayden000106 jayden000106 changed the title Feat: [#2] 컨텐츠 세부화면 UI 구성 [FEAT] #2 컨텐츠 세부화면 UI 구성 Jul 12, 2022
@commitcomplete
Copy link
Owner

일단 conflict나는 부분은 일시적으로 수정했습니다. 혹시 오류가 생기면 말씀해주세요

@commitcomplete commitcomplete merged commit 455e2a3 into commitcomplete:main Jul 12, 2022
Netflix CloneCoding automation moved this from In progress to Done Jul 12, 2022
@commitcomplete commitcomplete mentioned this pull request Jul 12, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@commitcomplete commitcomplete Awaiting requested review from commitcomplete

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
Netflix CloneCoding
Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@jayden000106 @commitcomplete