Ward-Group / Ward_iOS

0 stars 1 forks source link

Feature/notification sheet #46

Closed peppermint100 closed 6 months ago

peppermint100 commented 6 months ago

공통 뷰에서 버튼을 삭제한 이유는

  1. 홈 화면 -> 알림 화면 -> 알림 화면의 아이템 터치로 다른 화면 이동이 필요할 것으로 보임(지금 생각해보니 한번확인해볼걸 그랬나싶네요.)
  2. 이 경우 알림화면이 닫히고 홈화면에서 네비게이션 이동을 해야함
  3. 그러면 홈화면 내부의 네비게이션 헤더의 알림버튼이 홈화면으로 요청을 보내야함
  4. MVVM Input, Output의 패턴에서 양방향 이벤트 핸들링을 하게 됨

이해가 잘 가실지 모르겠네요. 일단 검색버튼, 알림버튼을 기능정의서에 따르면 홈화면에만 있기 때문에 홈화면에서 구축하고 시트를 홈화면에서 컨트롤하는게 좋아보입니다. 만약 네비게이션 헤더가 버튼을 가지고 있게 되면 홈화면 내부의 네비게이션 헤더가 홈화면으로 요청을 보내는 상황이 발생하여 Input Output 패턴의 단방향 방식이 깨지게 되어서 변경했습니다.

이후에 Router 패턴을 사용할 때에도 공통 네비게이션 바가 라우터를 가지게 된다면 역시 역으로 요청을 보내는 상황이 생길 것 같아요!