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

[FIX] #156 - 명함생성뷰 버그 해결 #157

Merged
merged 4 commits into from
Dec 16, 2021

Conversation

hyun99999
Copy link
Member

@hyun99999 hyun99999 commented Dec 16, 2021

🌴 PR 요약

🌱 작업한 브랜치

  • feature#156

🌱 작업한 내용

키보드

  • 뷰 클릭하면 키보드 다운.

피커뷰 완료버튼

  • 필수 정보 체크 기능 추가.
  • 필수, 옵션 정보 갱신 기능 추가.

명함배경 선택

  • 이미지 피커 선택 이후 셀 셀렉트 유지
  • 이미지를 선택한 직후에 뷰가 다시 등장할 때 0 인덱스를 셀렉트 하도록 함

birth, mbti

  • birth 와 mbti 텍스트필드 -> 텍스트뷰와 라벨로 구성 변경
  • 텍필 -> birth, mbti 이동시에 키보드 내려가지 않던 버그 해결
  • birth, mbti 수정 시 테두리 변경 추가
  • 바텀시트 hideBottomSheetAndGoBack 오버라이드해서 birth,mbti 바텀시트 닫힐 때 노티보내도록 함

브랜치 이동해보셔서... 정말 다른 버그는 없는지 봐주십시오.. 선배륌들 그래서 스샷은 없습니다.. ㅋ.ㅋ

📮 관련 이슈

- 뷰 선택하면 키보드 내려가도록 코드 추가
- birth 와 mbti 텍스트필드에서 텍스트뷰와 라벨로 구성 변경
- 텍필 -> birth, mbti 시에 키보드 내려가지 않던 버그 해결
- birth, mbti 수정 시 테두리 변경 추가
- 바텀시트 hideBottomSheetAndGoBack 오버라이드해서 birth,mbti 바텀시트 닫힐 때 노티보내도록 함
- 이미지 피커 이후에도 셀이 셀렉트가 풀리지 않도록 함
- 이미지를 선택한 직후에 뷰가 다시 등장할 때 0 인덱스를 셀렉트 하도록 함
- 피커뷰 완료 버튼 누를 때도 필수정보가 있는지 체크 기능 추가
- 피커뷰 완료 버튼 누를 때도 정보가 갱신되도록 기능 추가
@hyun99999 hyun99999 added Hyungyu 🐯 현규 교수님 작업 P1 / Priority High 우선순위 높음 labels Dec 16, 2021
@hyun99999 hyun99999 self-assigned this Dec 16, 2021
@hyun99999 hyun99999 added the Fix 코드 수정 및 버그, 오류 해결 label Dec 16, 2021
Copy link
Member

@dlwns33 dlwns33 left a comment

Choose a reason for hiding this comment

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

수고하셨습니당!

@@ -167,6 +172,10 @@ extension BackCardCreationCollectionViewCell {
}
}
}
@objc
private func dismissKeyboard() {
_ = textFieldList.map { $0.resignFirstResponder() }
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 Author

Choose a reason for hiding this comment

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

앞면에서 birth, mbti 바텀시트를 키게되면 뒷면에서 키보드를 불러왔더라도 꺼주는 역할을 하는 목적으로 만든 함수인데요!

코드는 map 함수를 통해서 텍스트필드리스트(뒷면의 텍스트필드가 들어있음) 안의 모든 텍스트필드의 키보드를 다운하는 함수입니다.

@hyun99999 hyun99999 merged commit 8da4424 into TeamNADA:develop Dec 16, 2021
@hyun99999 hyun99999 deleted the feature/#156 branch December 16, 2021 10:47
Comment on lines -40 to +46
@IBOutlet weak var birthTextField: UITextField!
@IBOutlet weak var mbtiTextField: UITextField!

@IBOutlet weak var birthLabel: UILabel!
@IBOutlet weak var birthView: UIView!
@IBOutlet weak var mbtiLabel: UILabel!
@IBOutlet weak var mbtiView: UIView!
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 Author

Choose a reason for hiding this comment

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

텍스트 필드에서 생년월일,mbti 텍스트 필드로 이동할때 textFieldDidEndEditing() 델리게이트 메서드에 textField.resignFirstResponder() 메서드를 추가해도 키보드가 내려가지 않아서 마지막 방법으로.. 뷰와 라벨로 바꾸었슴다

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix 코드 수정 및 버그, 오류 해결 Hyungyu 🐯 현규 교수님 작업 P1 / Priority High 우선순위 높음
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FIX] 명함생성뷰 버그 해결
3 participants