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

[FEAT] 주문 상세 페이지 구현 #92

Merged
merged 3 commits into from
Nov 28, 2024

Conversation

Taew00k
Copy link
Collaborator

@Taew00k Taew00k commented Nov 28, 2024

관련 이슈 🛰️

해결한 이슈 번호를 작성해주세요
close #88

작업 내용 🧑‍💻

작업한 내용을 간략히 작성해주세요

  • 주문 상세 컴포넌트 합치기
  • 컴포넌트 가운데 정렬 시키기

PR 포인트 🗯️

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

  • 기존에 만들어둔 컴포넌트들에 배경을 white로 적용을 안 했어서 불가피하게 이전 파일들 수정이 있었는데 새로 만든 파일 제외하고는 확인 안하셔도 됩니다!

알게된 점 🚀

기록하며 개발하기!

  • 컴포넌트들이 가운데 정렬되지 않아 어려워하던 중, 리드님께서 margin: 0 auto;를 사용해 보라고 알려주셨고, 덕분에 문제를 해결할 수 있었습니다. 이를 통해 요소를 가운데 정렬하려면 width를 지정한 뒤 margin: 0 auto;를 적용하면 간단하게 정렬할 수 있다는 점을 알게 되었습니다.
  • 성희님이 작성해주신 코드를 보면서 ProductLayout에서 children을 받아 메인 콘텐츠를 동적으로 넣는 방식을 적용해 볼 수 있었습니다.

참고 자료 📖 (선택)

참고했던 문서들 공유하기!

스크린샷 📸 (선택)

-.Clipchamp.7.mp4

@Taew00k Taew00k self-assigned this Nov 28, 2024
Copy link
Collaborator

@yarimu yarimu left a comment

Choose a reason for hiding this comment

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

LGTM❕❕많은 뷰 퍼블리싱 수고하셨습니다!!


export const orderLayoutContainerStyle = (theme: Theme) => css`
width: 100vw;
margin: 0 auto;
Copy link
Collaborator

Choose a reason for hiding this comment

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

성희님 pr에서 layout공통으로 만들까 고민하실때 뷰상 유사성이 없어서 안만들어도 된다고 생각했었는데, 이제보니 이런 방법이..! 리드ㄷ ㄷ

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

우리리드짱👍👍

Copy link
Collaborator

Choose a reason for hiding this comment

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

layout이 이렇게 공통으로 쓰일 수 있다는 게 완전히 와닿네요!!!

@bykbyk0401
Copy link
Collaborator

진짜 너무너무 수고하셨습니다ㅠㅠㅠ
가운데정렬도 아주 완벽하게 되네요👍
퍼블리싱하느라 매일 너무 고생많았습니당

Copy link
Collaborator

@seong-hui seong-hui left a comment

Choose a reason for hiding this comment

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

태욱이 혼자서 다 만들어버린 페이지 드디어 완성이라니 진짜 너무너무너무 고생 많으셨습니다!!!!!! 👍 👍👍 두 페이지밖에 없는데 혼자서 한페이지를 다 만들다니이ㅣ ㅣ,,,,정말로 넘넘 고생 많았고 너무 이쁘네요 진짜로 고생 많앗다,,

@Taew00k Taew00k merged commit 0e9025e into develop Nov 28, 2024
2 checks passed
@Taew00k Taew00k deleted the feat/#88/order-detail-page-publishing branch November 28, 2024 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] 주문 상세 페이지 구현
4 participants