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

[client] cart페이지 반응형 구현 #30

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

pki071120
Copy link
Collaborator

개요 💡

cart 반응형 구현

작업내용 ⌨️

반응형을 구현하여 폰을 사용할 때에도 디자인상 문제없이 사용할 수 있도록 제작하였습니다.
Screenshot 2024-09-25 at 3 16 51 PM

관련 issue 🤯

반응형 구현 작업을 처음 진행해보아서 많이 부족한 부분이 있을것 같습니다.

Copy link

vercel bot commented Sep 25, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
admin_bigbro_company ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 25, 2024 6:18am
client_bigbro_company ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 25, 2024 6:18am

Comment on lines +26 to +34
export const BottomBox = styled.div`
width: 100%;
height: 80%;
display: flex;
flex-direction: column;
align-items: flex-start;
gap: 1.25rem;
margin-top: 3.5rem;
`;
Copy link
Collaborator

Choose a reason for hiding this comment

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

스크린샷 2024-09-25 오후 3 45 04
해당 height가 80%인 것 때문에 특정 화면에서는 화면이 길게 나오는 현상이 있습니다.

@gjaegyun
Copy link
Collaborator

스크린샷 2024-09-25 오후 3 47 52
현재 space between + height 80% + margin top 때문에 선택 부분과 금액부분의 간격이 좀 긴 것 같습니다.

@gjaegyun
Copy link
Collaborator

현재 875px ~ 625px 부터 해당 사진 부분이 글자가 너무 클 뿐더러 어색하게 보입니다
스크린샷 2024-09-25 오후 3 50 27

@gaoooon
Copy link
Contributor

gaoooon commented Sep 26, 2024

지금 state를 각각 pc, mobile layout에서 생성해주고 있는데 root에다가 선언하고 props로 내려주는 방식으로 변경해야 할거 같습니다
그래야지 width가 바뀌었을때도 상태는 안바뀌어서 기존과 같은 상태로 보여집니다

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.

3 participants