Skip to content

Conversation

@Little-tale
Copy link
Collaborator

📝 Issue Number

  • NULL

🔧 변경 사항

  • 팝업 뒷배경 나오지 않는 문제 해결
  • 팝업 키보드 높이 부드럽게 올라오지 못하는 문제 해결
  • 버튼 액션 변경
  • 뷰 터치 제스처 Wrapper Class 생성 ( Object-C )
  • BasePopupViewController 생성 -> 현재 사용중은 아닙니다. 후의 분리하실때 애용 부탁드립니다.
  • ViewComponents init 속성 수정 함수 생성 -> "setOf"

🔍 변경 이유

  • 각 뷰들의 책임을 분리해야 합니다.
  • UI -> PresenterLayer 책임

🔗 참고 자료

  • NULL

🧐 추가 설명

  • NULL

Copy link
Collaborator

@seungyooooong seungyooooong left a comment

Choose a reason for hiding this comment

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

우선 리뷰가 늦어져 죄송합니다.

제 PR에 남겨주신 리뷰를 보고 수정을 한 다음에 리뷰를 봐서 충돌도 좀 많이 나고 살짝 코드가 꼬였는데 그것도.. 미리 확인을 못했어요 죄송해요 😓

그리고 완전히 이해를 못한 코드가 사실 좀 있는데 추가 설명을 약간 부탁드려도 될까요

댓글로 남겨주셔도 좋은데 가능하시다면 (질문이 한 번에 다 해결이 안되고 들으면서 계속 생길 것 같아서) 회의 때 말씀해주시면 감사하겠습니다!


nextButton = DogetherButton(action: updatePopup, title: "다음", status: .enabled)
let _nextButton = DogetherButton(title: "다음", status: .enabled, buttonTapped: updatePopup)
self.nextButton = _nextButton
Copy link
Collaborator

Choose a reason for hiding this comment

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

❓ : 임시 변수를 사용하시는 이유가 있나요?

self.nextButton = DogetherButton(title: "다음", status: .enabled, buttonTapped: updatePopup)

위 방식과 차이가 궁금합니다

}

// MARK: UI
extension PopupViewController {
Copy link
Collaborator

Choose a reason for hiding this comment

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

❓ : BasePopupViewController를 만드셨는데 여기서 사용하지 않으신 이유가 있나요?

  • BasePopupViewController에서 남긴 코멘트들을 여기에서도 적용해주세요!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

팝업콘트롤러가 모델을 물고있어서 다른코드들이랑 경합
상황이 날 가능성이 높아 해당 뷰컨담당 쪽네서 수정이 필요합니다

Copy link
Collaborator

Choose a reason for hiding this comment

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

제가 정확히 이해를 못한 것 같은데 말씀하시는 뷰컨담당 이라는 게 팝업뷰컨트롤러 담당을 말씀하신걸까요?

만약 그렇다면 팝업쪽 담당은 재형님이셨던건데.. 어떻게 된거죠?

import Foundation
import UIKit

@available(*, deprecated, message: "해당 메니저는 더이상 사용되지 않습니다.")
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍 : 아예 지워주셔도 좋을 것 같습니다!

@Little-tale
Copy link
Collaborator Author

팝업에 다 영향가서 참...

seungyooooong added a commit that referenced this pull request Jul 19, 2025
+ 인증 목록 필터의 경우 UI 일부 수정을 포함했어요
seungyooooong added a commit that referenced this pull request Jul 19, 2025
필터 선택 방식을 통일했어요
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants