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

[마이페이지,시작페이지] 최종 QA 반영입니다 ! #471

Merged
merged 9 commits into from
Mar 11, 2024

Conversation

hoeun0723
Copy link
Contributor

@hoeun0723 hoeun0723 commented Mar 10, 2024

이슈 넘버

구현 사항

  • 시작페이지 - 토너먼트 애니메이션 직접 구현 => 이부분 새롭게 이슈 팠습니다!
  • 마이페이지 - 종료된 선물방 엠티뷰 중앙정렬
  • 진행 중인 선물방 - 카드 컴포넌트 잘리는 부분 확인
  • 카드 컴포넌트 - 태그 1rem 정도 올려주기 (font 이슈)
  • 카드 컴포넌트 - 사람과 사람 수 위치 정렬

Need Review

  • 애니메이션 직접 구현 때문에 자꾸만 pr 이 늦어졌네요 !!! 해당 부분은 따로 디자인쌤께 png 요청하거나 lottie 라이브러리 자체를 수정해봐야 좋을 거 같아서, 애니메이션 파일만 만들어두고, lottie 컴포넌트 상태로 유지해뒀습니다 !! 해당 부분 참고하여 봐주시면 좋을 거 같아요 !!

📸 스크린샷

bandicam.2024-03-10.16-17-32-251.mp4

Reference

@hoeun0723 hoeun0723 self-assigned this Mar 10, 2024
@hoeun0723 hoeun0723 changed the title Refactor/#460 final qa [마이페이지,시작페이지] 최종 QA 반영입니다 ! Mar 10, 2024
Copy link

github-actions bot commented Mar 10, 2024

PR Preview Action v1.4.6
🚀 Deployed preview to https://SWEET-DEVELOPERS.github.io/sweet-client/pr-preview/pr-471/
on branch gh-pages at 2024-03-11 14:53 UTC

@hoeun0723 hoeun0723 added refactor ⚙️ 개선 사항 style🎨 작은 사항 수정 및 style 코드 작성 마이페이지 👤 마이 페이지 시작페이지 📄 스윗 맨 처음 진입 화면 페이지 labels Mar 10, 2024
@@ -3,6 +3,7 @@ import styled from 'styled-components';
export const DetailDoneRoomWrapper = styled.div`
${({ theme: { mixin } }) => mixin.flexCenter({})};
width: 100%;
height: 100vh;
Copy link
Member

Choose a reason for hiding this comment

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

오오...!! 스크롤 안되던 문제를 100vh를 줘서 해결하신걸까요??

Copy link
Contributor Author

Choose a reason for hiding this comment

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

앗 중앙정렬을 위해 100vh를 적용해주었습니다 !

@hoeun0723 hoeun0723 merged commit 07647eb into develop Mar 11, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor ⚙️ 개선 사항 style🎨 작은 사항 수정 및 style 코드 작성 마이페이지 👤 마이 페이지 시작페이지 📄 스윗 맨 처음 진입 화면 페이지
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants