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.

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

#220 : 신고 페이지 쪽지 수신함 위 노출하도록 변경해요. #222

Merged
merged 4 commits into from
Jan 20, 2025

Conversation

jeongjaino
Copy link
Member

@jeongjaino jeongjaino commented Jan 15, 2025

1. 📄 관련된 이슈 및 소개

#220 : 신고 페이지 쪽지 수신함 위 노출하도록 변경해요.

2. 🔥 변경된 점

기존 쪽지 신고 페이지가 하나의 Screen, Destination 단위였는데요 !

해당 페이지를 모달 형태로 구현했습니다.

변경이유

  • 화면 전환의 어려움
    • 신고페이지를 닫을 때, 이전 수신함 페이지로 전환해야 하는데, 이를 navigate로 구현하면 이전 신고화면이 백스택에 남게됨.
    • popBackStack을 활용하면, 수신함 페이지가 아닌 쪽지 홈으로 전환됨. (내부 상태에 따라 쪽지 수신함으로 가기 때문, 디폴트는 홈)
  • 기획을 가장 잘 반영한게 Dialog 형태인 것 같아 이로 변경하였습니다.

3. ✅ 필수 체크 사항

전체 Navigation내 Screen단위가 종료되어야 ViewModel이 소거되는데요.

ReportViewModel이 MessageReportScreen이 안보여도 상태가 남아있는 이슈가 있어,

모달을 열 때, 한번 상태를 초기화하고 열도록 변경했습니다.

4. 📸 작업물 사진 공유(선택)

  1. 기존 동작과 동일한지 확인했습니다. (쪽지 아이디 불러와서 API 콜 성공)
  2. 닫히는 로직 및 토스트 동작 확인하였습니다.

변경 전

Screen_Recording_20250115-205447_WeSpot.webm

변경 후

report1.webm

5. 💡알게된 혹은 궁금한 사항

Dialog 내 아래 속성을 적용하면, 전채 너비로 설정된다.

properties = DialogProperties(usePlatformDefaultWidth = false),

@jeongjaino jeongjaino self-assigned this Jan 15, 2025
@jeongjaino jeongjaino added 🍻제이콥🍻 24기 정진호 🔨픽스🔨 버그 및 잠재적 문제 해결 🔥리뷰 기다리는 중🔥 PR을 올리고 코드리뷰를 기다리고 있는 상태입니다. labels Jan 15, 2025
@jeongjaino jeongjaino linked an issue Jan 15, 2025 that may be closed by this pull request
1 task
Copy link
Contributor

@flash159483 flash159483 left a comment

Choose a reason for hiding this comment

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

위 아래로 움직이는 애니메이션이 없으니 보기 넘 좋은데 activity 분리 작업해야되나

@jeongjaino
Copy link
Member Author

위 아래로 움직이는 애니메이션이 없으니 보기 넘 좋은데 activity 분리 작업해야되나

오빠야 해줘

@jeongjaino jeongjaino merged commit f00b0f2 into develop Jan 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍻제이콥🍻 24기 정진호 🔥리뷰 기다리는 중🔥 PR을 올리고 코드리뷰를 기다리고 있는 상태입니다. 🔨픽스🔨 버그 및 잠재적 문제 해결
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WS-458 [FIX]: 신고 화면 쪽지 수신함 위 노출하기
2 participants