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.

Sign up for GitHub

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

Jump to bottom

[Design] 모각코 게시글 상세 페이지 퍼블리싱 #131

Merged
merged 14 commits into from
Nov 22, 2023
Merged

[Design] 모각코 게시글 상세 페이지 퍼블리싱 #131

merged 14 commits into from
Nov 22, 2023

Conversation

js43o
Copy link
Collaborator

@js43o js43o commented Nov 21, 2023 •
edited
Loading

설명

완료한 기능 명세

  • 모각코 게시글 페이지 레이아웃 작성
  • 모각코 게시글 페이지 스타일링

동작 화면

bandicam.2023-11-21.20-29-06-798.mp4

리뷰 요청 사항

특별히 리뷰해 주었으면 하는 부분, 고민되는 부분 기재하기

  • 댓글 UI는 기능 우선순위가 낮아 일단 제외하였습니다.
  • ChattingHeader 컴포넌트에서 사용되던 UserInfo 컴포넌트를 commons/UserChip 컴포넌트로 분리하였으며, 현재 페이지에서 참여자 목록을 표시할 때에도 사용했습니다.
  • 새 아이콘과 지도 샘플 이미지 파일을 asset으로 추가했습니다. [Fix] 메인이미지 경로 변경 #116 머지 후 파일 경로 다시 반영하겠습니다.
    • 위 두 가지 이유와 더불어 이름 변경, 코드 포맷팅 등의 작업으로 인해 Files changed가 상당히 많아졌습니다...😭 새로 추가된 파일 위주로 봐주시면 감사드리겠습니다..!
  • 모각코 게시글 상세 페이지는 일단 확인을 위해 /post 경로로 라우팅 해놓은 상태입니다. API 연동 후 /mogaco/:id 경로로 변경할 예정입니다.

@js43o js43o self-assigned this Nov 21, 2023
@js43o js43o force-pushed the add-mogaco-post-page branch from 52062ff to c82bb4c Compare November 21, 2023 09:01
@js43o js43o marked this pull request as ready for review November 21, 2023 09:29
@js43o js43o linked an issue Nov 21, 2023 that may be closed by this pull request
[FEATURE] 모각코 게시글 상세 페이지 읽기 구현 #123
Closed
2 tasks
ttaerrim
ttaerrim reviewed Nov 21, 2023
View reviewed changes
app/frontend/src/hooks/useRouter.tsx Outdated Show resolved Hide resolved
app/frontend/src/pages/Post/index.tsx Outdated Show resolved Hide resolved
app/frontend/src/pages/Post/index.tsx Outdated Show resolved Hide resolved
app/frontend/src/pages/Post/index.tsx Outdated Show resolved Hide resolved
LEEJW1953
LEEJW1953 reviewed Nov 21, 2023
View reviewed changes
Copy link
Collaborator

@LEEJW1953 LEEJW1953 left a comment

Choose a reason for hiding this comment

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

작성 페이지 퍼블리싱도 MogacoPost 라는 이름이 어울릴 것 같아서 그렇게 작업하고 있었는데, 상세 페이지와 겹칠 것 같아서 둘 중 하나를 수정해야 할 것 같습니다
다른분들의 아이디어가 궁금합니다..

app/frontend/src/pages/Post/index.tsx Outdated Show resolved Hide resolved
js43o
js43o commented Nov 22, 2023
View reviewed changes
Copy link
Collaborator Author

@js43o js43o left a comment •
edited
Loading

Choose a reason for hiding this comment

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

글 작성 페이지: WritePage, MogacoWrite
상세 게시글 페이지: PostPage, MogacoPost
이렇게 나누는 건 어떨까요?

=> 작성(MogacoPost), 상세(MogacoDetail)로 결정

@js43o js43o force-pushed the add-mogaco-post-page branch from 0930c5a to c18bb8a Compare November 22, 2023 02:12
@js43o
🍱 새로운 아이콘 추가
bded687 
- 크기가 24px인 아이콘에 '_large' 접미사 추가
@js43o js43o force-pushed the add-mogaco-post-page branch from 360457d to 565a586 Compare November 22, 2023 02:40
@js43o js43o merged commit 63def5c into boostcampwm2023:develop Nov 22, 2023
1 check passed
@js43o js43o deleted the add-mogaco-post-page branch November 22, 2023 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LEEJW1953 LEEJW1953 LEEJW1953 approved these changes

@ttaerrim ttaerrim ttaerrim approved these changes

Assignees

@js43o js43o

Labels
💄 design 🎨 Front-end
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[FEATURE] 모각코 게시글 상세 페이지 읽기 구현
3 participants
@js43o @ttaerrim @LEEJW1953