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.

Sign up for GitHub

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

Jump to bottom

Feat/#62 - 체크박스와 사용 버튼 연동. 근데 이제 약간의 오류를 곁들인 #65

Merged
merged 3 commits into from
Nov 29, 2024
Merged

Feat/#62 - 체크박스와 사용 버튼 연동. 근데 이제 약간의 오류를 곁들인 #65

merged 3 commits into from
Nov 29, 2024

Conversation

OneTen19
Copy link
Contributor

🔗 연결된 이슈

📄 작업 내용

구현 내용 IPhone 13 mini
GIF

💻 주요 코드 설명

  • 클로저를 이용해서 구현했습니다
var checkboxTappedHandler: ((Bool) -> Void)?
cell.checkboxTappedHandler = { [weak self] isChecked in
                guard let self = self else { return }

                if isChecked {
                    self.secondSectionCheckedStatus.insert(indexPath)
                } else {
                    self.secondSectionCheckedStatus.remove(indexPath)
                }

                self.updateUsingButton()
            }

👀 기타 더 이야기해볼 점

Simulator Screen Recording - iPhone 13 mini - 2024-11-29 at 02 36 38

갤러리 뷰에서 사진을 선택해서 디테일 뷰에서 체크박스를 선택한 후에 나오면,
갤러리 뷰에서 체크박스를 다시 선택해서 지워도 사용 버튼이 상시 활성화 되어 있어요.

갤러리에서 체크를 해제한 후에 해당 사진을 클릭해서 한 번 더 들어갔다가 나오면 다시 정상적으로 작동하는데..
뭔가 연결이 꼬여있는 느낌이긴 한데 도무지 모르겠음..

업데이트가 편리한 DiffableDateSource를 어려워도 공부해서 사용하는 게 더 편하고 깔끔한 길이라는 생각이 드네요

@OneTen19 OneTen19 added Feat 기능 구현시 사용 Hanyeol 김한열김한열김한열김한열김한열김한열 labels Nov 28, 2024
@OneTen19 OneTen19 self-assigned this Nov 28, 2024
@OneTen19 OneTen19 linked an issue Nov 28, 2024 that may be closed by this pull request
Feat: Connect Using Button with Checkbox #62
Closed
1 task
hooni0918
hooni0918 approved these changes Nov 29, 2024
View reviewed changes
Copy link
Collaborator

@hooni0918 hooni0918 left a comment

Choose a reason for hiding this comment

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

👍

yurim830
yurim830 approved these changes Nov 29, 2024
View reviewed changes
Copy link
Contributor

@yurim830 yurim830 left a comment

Choose a reason for hiding this comment

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

어렵다어려워~~ㅠㅠ 고생하셨습니다!

HEHEEUN
HEHEEUN approved these changes Nov 29, 2024
View reviewed changes
Copy link
Contributor

@HEHEEUN HEHEEUN left a comment

Choose a reason for hiding this comment

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

고생많았으!!

@OneTen19 OneTen19 merged commit a5f850a into yurim Nov 29, 2024
@OneTen19 OneTen19 deleted the Feat/#62-UsingButton branch November 29, 2024 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@HEHEEUN HEHEEUN HEHEEUN approved these changes

@hooni0918 hooni0918 hooni0918 approved these changes

@yurim830 yurim830 yurim830 approved these changes

Assignees

@OneTen19 OneTen19

Labels
Feat 기능 구현시 사용 Hanyeol 김한열김한열김한열김한열김한열김한열
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Feat: Connect Using Button with Checkbox
4 participants
@OneTen19 @HEHEEUN @hooni0918 @yurim830