Fix[#80, #81], Feat[#98]: 팝오버와 관련된 자잘한 버그를 해결하고, 레이아웃을 개선했습니다.

[seongkyun-lim]

Motivation

• 사용자가 콩받을 만한 자잘한 버그들이 있었습니다.
• 레이아웃 조정에 관한 @LeeSungNo-ian 의 요청이 있었습니다.

Key Changes

Feat

• 약속을 만들 때 제목이 비어있으면 완료 버튼 비활성화.

  Fix

• 팝오버 memo 편집창에서 placeholder 부분이 눌리지 않던 문제 해결.
• 약속을 삭제하는 순간에 셀의 제목과 메모가 "이름 없음", "내용 없음"으로 바뀌는 문제 해결.

  Layout

• 팝오버 창 레이아웃 변경
  • 팝오버 네비게이션 바 상단 영역 줄임.
  • 네비게이션 바, 내용&메모 편집 부분, 날짜 편집 부분 사이의 간격을 좁힘.
  • 아이패드 12.9인치, 11인치, 미니에서 팝오버 창이 가려지지 않도록 수정.

Results

[LeeSungNo-ian]

좋습니다 굿~!

[seongkyun-lim]

팀을 다시 등록하라고 해서 기존에 설정되어 있던 팀을 다시 넣었는데, .pbxproj 파일에서 해당 부분에 conflict가 발생했습니다.

변경된 팀 코드를 선택해서 해결했는데, 1) 왜 같은 팀으로 설정했는 코드가 바뀌었는지, 2) 다른 분들 빌드하는 데 영향을 미치지 않을지 궁금합니다.

[Valentino1994]

누군가 팀코드를 처음 넣었고 그 이후로 각자 Pull 하고 나서 팀을 개인으로 바꿔줄 필요가 생겼슴니다.

[seongkyun-lim]

누군가 팀코드를 처음 넣었고 그 이후로 각자 Pull 하고 나서 팀을 개인으로 바꿔줄 필요가 생겼슴니다.

@Valentino1994 그러면 앞으로 PR할 때마다 충돌이 나는 건가요?

[Valentino1994]

@seodam-hst 아마도 저번에 vivi가 보여줬던 프로세스를 거치지 않으면 계속 그럴거예요.

[seongkyun-lim]

kko_okk/Views/BoardViews/TodoTab/ButtonForContract.swift

Text(contract.memo ?? "내용 없음")  // contract의 memo(하단 자세한 내용)을 받아와서 보여줌
Text(contract.memo ?? "")  // contract의 memo(하단 자세한 내용)을 받아와서 보여줌

name하고 memo가 안들어가 있을 확률이 있긴 한데 그냥 빈 텍스트로 두는 게 좋을까요 ? 아니면 nil이라도 두는게 좋을까요 ? 아니면 save할 때 name하고 memo가 없으면 저장이 안되도록 하는게 좋을까요 ?

CoreData에 저장할 때 저 값을 직접 사용하지 않기 때문에 nil로 둘 필요는 없어 보여요! memo가 필수 입력값이면 너무 귀찮지 않을까요?