Skip to content

Conversation

@lee-yeonwoo
Copy link
Collaborator

@lee-yeonwoo lee-yeonwoo commented Jun 1, 2023

🌱 작업한 내용

  • 편집 모드 종료시 , 컬렉션 뷰를 reload() 하여 선택셀 모드가 초기화 되도록 하게 했습니다
  • 06.02 추가 사항⬇️
  • 업로드한 코스 뷰, 내가 그린 코스 뷰에 있는 셀에 체크박스가 뜨도록 UI를 수정했습니다
  • 편집모드에 따라 체크박스의 여부가 달라지도록 구현했습니다
  • 업로드한 코스 뷰에서 편집을 취소할 시, 편집모드가 종료되지 않는 부분을 수정하여 편집을 취소하면 원래 상태로 돌아가도록 구현했습니다

🌱 PR Point

  • 다른..방법도 있으려나요..?
  • 06.02 추가 PR POINT : 컬렉션뷰 내에 있는 함수들도 따로 빼면 좋을 것 같죠 ...? 근데 그러면 함수의 위치를 위로 올려도 ... 괜찮나요?그게 더 보기 편할런지 등등 업로드한 코스 부분 중점 PR 주시면 감사하겠습니다 !

📸 스크린샷

Simulator Screen Recording - iPhone 14 Pro - 2023-06-02 at 13 37 36

📮 관련 이슈

Copy link
Contributor

@dlwogus0128 dlwogus0128 left a comment

Choose a reason for hiding this comment

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

멋져용

Copy link
Collaborator

@lsj8706 lsj8706 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~!

Copy link
Collaborator

@lsj8706 lsj8706 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~!

@lee-yeonwoo lee-yeonwoo changed the title [Fix] #163 - editmode 종료시 reload() 할 수 있도록 수정 [Fix] #163 - editmode 종료시 reload() 할 수 있도록 수정 Jun 2, 2023
Copy link
Collaborator

@lsj8706 lsj8706 left a comment

Choose a reason for hiding this comment

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

리뷰 반영 감사합니다~!!

$0.backgroundColor = .w1
}

private let selectIdicatorButton = UIButton(type: .custom).then {
Copy link
Collaborator

Choose a reason for hiding this comment

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

오타가 있었네요 selectIndicatorButton 입니다..!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

오 내가 모르는 용어인줄...고쳤습니닷

Comment on lines 182 to 183
self.UploadedCourseInfoCollectionView.reloadData()
isEditMode.toggle()
Copy link
Collaborator

Choose a reason for hiding this comment

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

토글을 먼저 하고 reloadData()를 하는게 더 좋을 것 같아요!
그래야 celForItemAt에서 isEditMode에 알맞게 cell을 생성할 것 같습니다~~

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

넵!!

@lee-yeonwoo lee-yeonwoo merged commit 3452f4e into Runnect:develop Jun 2, 2023
@lee-yeonwoo lee-yeonwoo deleted the #163-보관함편집취소에러해결 branch June 2, 2023 05:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants