Skip to content

Feature/list, todayViewController에 모달추가 - #55

Merged
robolovo merged 4 commits into
masterfrom
feature/list
May 6, 2020
Merged

robolovo merged 4 commits into
masterfrom
feature/list

Conversation

@robolovo

@robolovo robolovo commented May 6, 2020

Copy link
Copy Markdown
Collaborator

금일 작업분 코드리뷰 요청드립니다.

  1. 앱 전체적인 디자인에 맞춰 리스트 디자인도 바꿔주었습니다.
  2. 리스트에서 셀 클릭시에 Today.storyboard가 아닌 Display.storyboard로 화면이동 되도록 변경하였고, displayViewController에 dateToSet 변수를 설정하여 화면이동시에 셀의 날짜값을 전달해주도록 구현하였습니다.
  3. todayViewController에서 이미지픽 버튼을 누르면 모달창이 띄어지고 사용자가 갤러리에서 사진을 골라 배경화면을 임의로 변경할 수 있는 기능을 추가해주는 코드를 구현하였습니다.

@robolovo
robolovo requested a review from 365kim May 6, 2020 09:21
@robolovo robolovo self-assigned this May 6, 2020
@365kim

365kim commented May 6, 2020

Copy link
Copy Markdown
Member

아니........ 오늘 회의 마친지 3시간도 안지났는데............?????????? 😮
그새 이걸 다 하셨다구요......................................? 😮
몸이 3개이신건가요....???............ 😮

@365kim

365kim commented May 6, 2020

Copy link
Copy Markdown
Member

사진 선택 너무 잘됩니다...감동입니다...

@365kim 365kim left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

모서리
리스트뷰로 넘어왔는데 모서리가 잘리는 문제가 있어요!
확인부탁드릴게요~~

@365kim
365kim requested a review from dev-jko May 6, 2020 10:09

@dev-jko dev-jko left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

잘 작동합니다

@robolovo
robolovo requested a review from 365kim May 6, 2020 10:48

@365kim 365kim left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

확인했습니다 고생하셨습니다 :)

func imagePickerController(_ picker: UIImagePickerController, didFinishPickingMediaWithInfo info: [UIImagePickerController.InfoKey : Any]) {
if let pickedImage = info[UIImagePickerController.InfoKey.originalImage] as? UIImage {
backgroundImage.contentMode = .scaleAspectFill
backgroundImage.contentMode = .scaleToFill

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

.scaleToFill은 이미지 가로세로 비율이 유지되지 않아서
.scaleAspectFill 로 채우는게 더 예쁠것같아요!

@robolovo
robolovo merged commit 1e5929b into master May 6, 2020
@robolovo
robolovo deleted the feature/list branch May 6, 2020 10:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants