책 편집하는 뷰 작성 #71
책 편집하는 뷰 작성 #71
Conversation
| private func configureNavigationBar() { | ||
| navigationController?.navigationBar.isHidden = true | ||
| } |
There was a problem hiding this comment.
P3: 생각해보니 저희 디자인엔 없지만.. 편집이 아니라 책을 새로 만드는 경우에는 뒤로 가기를 통해 책 표지 생성 페이지로 갈 수 있어야하지 않았나요...? 디자인 수정을 못해서 죄송합니다.. ㅠㅠ 그냥 네비바를 놔두고 좌상단엔 뒤로가기, 우상단에 발생을 넣는건 어떨까.. 싶습니당..
There was a problem hiding this comment.
좋습니당!! 저도 사실 불편하다고 생각하고 있었어욤
There was a problem hiding this comment.
처음 만들었을 때는 아래의 예쁜 발행버튼으로 나오기로 했습니다.
홈에서 들어갈 때는 오른쪽위에 저장버튼이 나오도록 했습니다!
사실 alert도 저장이 어떻게 되냐에 따라 달라지기는 하는 것같긴 합니다...
2024-11-21.1.32.00.mov
There was a problem hiding this comment.
그럼 이 화면에서는 네비 오른쪽 버튼과 타이틀은 없는 걸까요 ??
There was a problem hiding this comment.
그럼 이 화면에서는 네비 오른쪽 버튼과 타이틀은 없는 걸까요 ??
홈화면에서 누를경우에는 오른쪽위에 저장버튼 나오는 식으로 구성할 생각입니다! (일단 그렇게 코딩은 해놓음)
책정보에서 title가져오는 것은 좋을 듯 싶습니다! <- TODO로 남겨놓을게욤!
| NotificationCenter.default.addObserver( | ||
| self, | ||
| selector: #selector(keyboardWillAppear), | ||
| name: UIResponder.keyboardWillShowNotification, | ||
| object: nil | ||
| ) | ||
| NotificationCenter.default.addObserver( | ||
| self, | ||
| selector: #selector(keyboardWillHide), | ||
| name: UIResponder.keyboardWillHideNotification, | ||
| object: nil | ||
| ) |
There was a problem hiding this comment.
P1: NotificationCenter에 addObserver를 해줄경우 해당 VC가 deinit될 때, observing하고 있는 것을 remove해줘야합니다 ! remove코드도 제 editPhotoViewController에 존재하니 확인해보시고 추가해주시면 좋을 것 같습니다 !
There was a problem hiding this comment.
제가 알기로는 이 방식으로 추가해준 것은 따로 removeObserver를 안해주어도 되는 것으로 압니다! 문서링크
If your app targets iOS 9.0 and later or macOS 10.11 and later, you do not need to unregister an observer that you created with this function. If you forget or are unable to remove an observer, the system cleans up the next time it would have posted to it.
하지만, 통일성을 위해서 deinit에 넣어놓겠습니다! (혹시 제가 아는 것과 다른 점이 있으면 알려주세용!)
There was a problem hiding this comment.
전에 다른 멘토님께 remove 자동으로 된다고 들었는데, (자료 찾아보겠습니다)
꼭 remove 명시해줘야 한다는 자료가 있나요 ??
There was a problem hiding this comment.
엇 전에 제가 봤던 자료가 정현님이 올리신 거입니다
근데 저럴거면 없애는 걸로 통일하는게 좋지 않나요 ?
There was a problem hiding this comment.
옹 저는 remove를 해줘야한다는 것까지만 읽고 그 뒤에 iOS 9 이상부터는 안해도 된다는 부분을 안 읽었던 것 같습니다...ᐟ.ᐟ 그럼 제 코드에서 remove 부분을 삭제하는게 맞는 것 같네용 .ᐟ.ᐟ 정현님 번거롭게 해드려 죄송합니다 ㅠㅠ
There was a problem hiding this comment.
엇 전에 제가 봤던 자료가 정현님이 올리신 거입니다 근데 저럴거면 없애는 걸로 통일하는게 좋지 않나요 ?
둏습니다!
| // MARK: - UITableViewDataSource | ||
| extension EditBookViewController: UITableViewDataSource { | ||
| func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int { | ||
| return 10 |
There was a problem hiding this comment.
P3: 해당 부분은 dummy data이고 아마 viewModel의 page 개수를 가져올 자리인 것 같은데 TODO를 남겨두거나 viewModel.page.count로 수정하는건 어떨까요??
| return output.eraseToAnyPublisher() | ||
| } | ||
| private func fetchPages() { | ||
| pages = ["Image", "Video", "Text", "Audio"] |
There was a problem hiding this comment.
P3: 해당 부분도 TODO로 dummy data임을 표시해주거나 삭제해주는게 좋을 것 같습니다 !
| // MARK: - TouchEvent | ||
| override func touchesBegan(_ touches: Set<UITouch>, with event: UIEvent?) { | ||
| textView.becomeFirstResponder() | ||
|
|
||
| super.touchesBegan(touches, with: event) | ||
| } |
There was a problem hiding this comment.
잘 몰라서 그런데 얘는 무슨 동작을 하고 어떨 때 쓰이나요?!
There was a problem hiding this comment.
이게 그냥 TableViewCell에 TextView를 넣고 누르면 TableViewCell이 먼저 반응해서 TextView가 동작을 안하더라구요... 그래서 TableViewCell에서 온 터치 이벤트를 TextView에게 �알려주어서 키보드를 띄운다음에 didSelectRowAt이 되도록 super로 후속이벤트를 처리하게 했습니다!
There was a problem hiding this comment.
감사합니다 이 메소드를 잘 몰랐어서 궁금했습니다..!
responder chain 개념에 대한 코드 이군요
제가 더 알아보겠습니다 !!!
There was a problem hiding this comment.
감사합니다 이 메소드를 잘 몰랐어서 궁금했습니다..! responder chain 개념에 대한 코드 이군요 제가 더 알아보겠습니다 !!!
혹시 찾아보시다가 더 좋은방법!이 있다면 공유 부탁드립니다 ㅎㅎ헿
| NotificationCenter.default.addObserver( | ||
| self, | ||
| selector: #selector(keyboardWillAppear), | ||
| name: UIResponder.keyboardWillShowNotification, | ||
| object: nil | ||
| ) | ||
| NotificationCenter.default.addObserver( | ||
| self, | ||
| selector: #selector(keyboardWillHide), | ||
| name: UIResponder.keyboardWillHideNotification, | ||
| object: nil | ||
| ) |
There was a problem hiding this comment.
전에 다른 멘토님께 remove 자동으로 된다고 들었는데, (자료 찾아보겠습니다)
꼭 remove 명시해줘야 한다는 자료가 있나요 ??
| // MARK: - Keyboard Appear & Hide | ||
| @objc private func keyboardWillAppear(_ notification: Notification) { | ||
| guard let keyboardInfo = notification.userInfo?[UIResponder.keyboardFrameEndUserInfoKey], | ||
| let keyboardSize = keyboardInfo as? CGRect else { return } | ||
| let bottomConstant = -(keyboardSize.height - view.safeAreaInsets.bottom + 10) | ||
| buttonStackViewBottomConstraint?.constant = bottomConstant | ||
| UIView.animate(withDuration: 0.3) { [weak self] in | ||
| self?.view.layoutIfNeeded() | ||
| } | ||
| } | ||
| @objc private func keyboardWillHide() { | ||
| buttonStackViewBottomConstraint?.constant = Self.buttonBottomConstant | ||
| UIView.animate(withDuration: 0.3) { [weak self] in | ||
| self?.view.layoutIfNeeded() | ||
| } | ||
| } | ||
| } |
There was a problem hiding this comment.
오 이게 키보드 활성화 됐을 때 뷰 바닥이 키보드 위로 간다는 거군요
역시 대영현
| NotificationCenter.default.addObserver( | ||
| self, | ||
| selector: #selector(keyboardWillAppear), | ||
| name: UIResponder.keyboardWillShowNotification, | ||
| object: nil | ||
| ) | ||
| NotificationCenter.default.addObserver( | ||
| self, | ||
| selector: #selector(keyboardWillHide), | ||
| name: UIResponder.keyboardWillHideNotification, | ||
| object: nil | ||
| ) |
There was a problem hiding this comment.
옹 저는 remove를 해줘야한다는 것까지만 읽고 그 뒤에 iOS 9 이상부터는 안해도 된다는 부분을 안 읽었던 것 같습니다...ᐟ.ᐟ 그럼 제 코드에서 remove 부분을 삭제하는게 맞는 것 같네용 .ᐟ.ᐟ 정현님 번거롭게 해드려 죄송합니다 ㅠㅠ
#️⃣ 연관된 이슈
⏰ 작업 시간
편집셀하고 테이블뷰 관련 내용이 합쳐져 있어서 시간계산이 애매하네요
위에가 원래 시간이긴한데,일단 테이블뷰 만들기를 가져가고 편집셀에서는 반정도 가져오겠습니다.
평균: 3+2.3 = 5.3
본인: 3.5+4 = 7.5
실제 투자한 시간은 3.5정도입니다... 그동안 졸프때매 질질끌었네요... 죄송합니다 ㅠㅠ
📝 작업 내용
📸 스크린샷
2024-11-21.11.42.24.mov
📒 리뷰 노트
ViewModel에 대한 input-output 구성
어떻게 할지 살짝 고민입니다.
발행을 눌렀을 때 저장을 하게 하면 셀이 재사용되면서 내부 내용을 지워질것같다? 라는 느낌이 들어서...
사실 저장이 주요로직이다보니... 레포가 불확정적이라 문제네요 일단 다음 목표는 사진, 동영상 등등을 표현할 수 있는 뷰를 만드는 것을 목표로 잡겠습니다.