Skip to content

feat: 글 상세 페이지에 대한 디자인 시스템 적용#182

Merged
MTtankkeo merged 2 commits into
mainfrom
post_detail
Jul 2, 2026
Merged

feat: 글 상세 페이지에 대한 디자인 시스템 적용#182
MTtankkeo merged 2 commits into
mainfrom
post_detail

Conversation

@MTtankkeo

Copy link
Copy Markdown
Member

작업 내용

글 상세 페이지 전반에 GDS 기반 디자인 시스템을 적용했습니다용.

  • 게시글 제목, 작성자, 타입 칩, 공유/더보기 액션 UI 개선
  • 삭제 확인 UI를 기존 GrimityDialog에서 GdsAlert로 변경
  • 게시글 유틸바 및 공통 유틸바 아이콘/메뉴 동작 정리
  • 최신 글 영역을 기존 전용 provider에서 게시판 post provider 기반으로 변경
  • 댓글 입력 포커스 관련 불필요 상태 제거

피그마 참고

@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 post detail presentation layer to adopt GDS (Grimity Design System) components, simplifying layout logic, removing custom app bars, and replacing dialogs with standard alerts. Feedback highlights critical issues where unauthenticated users could be incorrectly identified as authors (isMine evaluating to true) when author information is missing in PostContentView, FeedUtilBar, and PostUtilBar. Additionally, the bookmark item in GrimityUtilBar is incorrectly bound to 'like' actions instead of 'save' actions, and PostLatestView contains a Riverpod anti-pattern by using ref.read inside the build method.

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/post_detail/view/post_content_view.dart
Comment thread lib/presentation/feed_detail/widget/feed_util_bar.dart
Comment thread lib/presentation/post_detail/widget/post_util_bar.dart
Comment thread lib/presentation/common/widget/grimity_util_bar.dart
Comment thread lib/presentation/post_detail/view/post_latest_view.dart Outdated
@MTtankkeo MTtankkeo merged commit e915849 into main Jul 2, 2026
1 check passed
@MTtankkeo MTtankkeo deleted the post_detail branch July 7, 2026 17: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.

1 participant