Skip to content

feat: 신고 페이지에 대한 디자인 시스템 적용#178

Merged
MTtankkeo merged 2 commits into
mainfrom
report
Jun 24, 2026
Merged

feat: 신고 페이지에 대한 디자인 시스템 적용#178
MTtankkeo merged 2 commits into
mainfrom
report

Conversation

@MTtankkeo

Copy link
Copy Markdown
Member

작업 내용

  • 신고 페이지에 GDS 디자인 시스템을 적용
  • 기존 신고 전용 AppBar, 전송 버튼, 성공 Dialog를 제거하고 GdsScaffold, GdsModal, GdsAlert, GdsListItem, GdsTextArea 기반으로 재구성
  • 모바일은 신고 페이지로 이동하고, 데스크톱은 모달로 신고 화면을 표시하도록 ReportPage.push 진입 방식을 추가
  • 신고 사유 선택, 상세 내용 입력, 닫기/신고하기 액션 영역을 분리하고 신고 가능 여부 로직을 provider로 정리
  • 신고 전 확인 Alert 추가
  • 신고 후 완료 Alert 추가
  • 피드/댓글/채팅 등 기존 신고 진입 지점을 ReportPage.push 사용 방식으로 변경

피그마 참고

@MTtankkeo MTtankkeo requested a review from aiden30015 June 23, 2026 12:39

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request refactors the report feature to support both modal and full-page presentations using the updated GDS design system components, while removing deprecated alert properties and old report widgets across various pages. Feedback on the changes highlights a bug in ReportBodyView where the title parameter is ignored in favor of a hardcoded string, a potential usability issue where NeverScrollableScrollPhysics blocks scrolling inside modals, and an opportunity to improve validation by trimming whitespace from the report content.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread lib/presentation/report/view/report_body_view.dart
Comment thread lib/presentation/report/view/report_body_view.dart
Comment thread lib/presentation/report/provider/report_provider.dart Outdated

@aiden30015 aiden30015 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

수고하셨습니다

@MTtankkeo MTtankkeo merged commit 0898f36 into main Jun 24, 2026
1 check passed
@MTtankkeo MTtankkeo deleted the report branch June 24, 2026 19:22
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.

2 participants