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] #68 - 스톱워치 + 사진 인증하기 중 사진인증 뷰 구현 #86

Merged
merged 9 commits into from
Jan 17, 2022

Conversation

yangsubinn
Copy link
Member

🌴 PR 요약

🌱 작업한 브랜치

🌱 작업한 내용

  • UI 함수 분리
  • 사진 이미지 뒤에 투명 uiview 추가
  • 스톱워치 -> 사진 인증 플로우 연결

📌 참고 사항

uploadImageView가 조금 이상한 친구입니다.
사진을 가져왔을때 aspectToFill이 적용이 안돼서
uploadImageView.layer.masksToBounds = true
이걸 일단 적용시켰고 제대로 작용을 하는 상황입니다

📸 스크린샷

기능 스크린샷
촬영 업로드 ezgif com-gif-maker (34)
선택 업로드
그냥 앨범 가져오는거냐 촬영이냐만 다른데 gif가 너무 느리고 영상도 커서 안올라가길래
그냥 냅다 이미지만 올려봅니다

📮 관련 이슈

@yangsubinn yangsubinn added Feat 새로운 기능 구현 🦹t없e맑은水빈 우리 선배 이젠 타이니하지 않아 P1 / Priority High labels Jan 17, 2022
@yangsubinn yangsubinn self-assigned this Jan 17, 2022
@yangsubinn yangsubinn changed the title Feature/#68 [Feat] #68 - 스톱워치 + 사진 인증하기 중 사진인증 뷰 구현 Jan 17, 2022
@yangsubinn yangsubinn added 🚨review_required🚨 긴급! 빠르게 확인해야할 사항 and removed P1 / Priority High labels Jan 17, 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.

훈훈한 코드 잘 보고 갑니다

Comment on lines +233 to +235
func imagePickerControllerDidCancel(_ picker: UIImagePickerController) {
self.dismiss(animated: true, completion: nil)
}
Copy link
Member

Choose a reason for hiding this comment

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

👍

Comment on lines +302 to +303
make.bottom.equalTo(view.safeAreaLayoutGuide).inset(10)
make.height.equalTo(self.view.frame.width*48/335)
Copy link
Member

Choose a reason for hiding this comment

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

오호 좋네여

Copy link
Member

@L-j-h-c L-j-h-c left a comment

Choose a reason for hiding this comment

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

플로우별로 함수 뺀거 깔끔한것같아요! 66따봉 드립니다.

@@ -55,7 +90,7 @@ extension AuthUploadVC {
retakeButton.backgroundColor = .sparkWhite
retakeButton.tintColor = .sparkDarkPinkred
retakeButton.setTitleColor(.sparkDarkPinkred, for: .normal)
retakeButton.layer.borderWidth = 1
retakeButton.layer.borderWidth = 2
Copy link
Member

Choose a reason for hiding this comment

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

수정감삼다!

@yangsubinn yangsubinn merged commit 490c676 into TeamSparker:develop Jan 17, 2022
@yangsubinn yangsubinn deleted the feature/#68 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
Feat 새로운 기능 구현 🚨review_required🚨 긴급! 빠르게 확인해야할 사항 🦹t없e맑은水빈 우리 선배 이젠 타이니하지 않아
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] 스톱워치 + 사진 인증하기 중 사진인증 뷰 구현
3 participants