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

Feature/#91: 대기방 리스트 화면 UI 구현 #119

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

Conversation

minjeongss
Copy link
Member

@minjeongss minjeongss commented Feb 5, 2025

#️⃣ 연관된 이슈

#91

📝 작업 내용

대기방 리스트 화면 UI를 구현했습니다. 구체적인 구현은 다음과 같습니다.

  • 대기방 리스트 화면 전용 commonPage의 pageWrapper 생성
  • 호스트의 캐릭터를 반영하여 배경 설정
  • 기존에 문제가 되었던 grid 레이아웃 재설정
    • 피그마에 존재하는 카드 컴포넌트의 width 무시
    • 피그마에 존재하는 화면의 좌우 padding 값, 카드 컴포넌트 사이 간격 반

📸 스크린샷

화면 레이아웃
image image

💬 리뷰 요구사항

  • 디자이너님께 배경화면을 받은 후, 변경된 배경화면 적용은 API 연결 파트에서 진행할 예정입니다.
  • 캐릭터의 타입이 5가지라고 알고 있는데, 더 존재한다면 알려주세요!
    • 현재 대기방 리스트에서 반영할 수 있는 호스트 캐릭터 정보
    import BasicIcon from '@styles/Icon/waitingRoomList/basic.png';
    import AngularIcon from '@styles/Icon/waitingRoomList/angular.png';
    import BumpyIcon from '@styles/Icon/waitingRoomList/bumpy.png';
    import DotIcon from '@styles/Icon/waitingRoomList/dot.png';
    import WoolIcon from '@styles/Icon/waitingRoomList/wool.png';
    

@minjeongss minjeongss self-assigned this Feb 5, 2025
@minjeongss minjeongss linked an issue Feb 5, 2025 that may be closed by this pull request
1 task
Comment on lines +5 to +10
type RefreshIconProps = { onClick: () => void };
export const RefreshIcon = ({ onClick }: RefreshIconProps) => (
<div onClick={onClick}>
<Image src={RefreshIconSrc} alt="Refresh Icon" />
</div>
);
Copy link
Member Author

Choose a reason for hiding this comment

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

Next의 Image는 onClick 이벤트가 직접적으로 등록이 되지 않아서, 상위 컴포넌트로부터 props을 onClick을 받아 div에서 처리하도록 설정했습니다.
Image의 Icon을 활용하시는 분들에게 도움이 되지 않을까 싶어 comment 추가적으로 남깁니다. 😁

@aaahyesu
Copy link
Contributor

aaahyesu commented Feb 5, 2025

기존 피그마 디자인과 비교했을때, 캐릭터 위치는 반영이 안된 것 같아요 🥹

화면 시안
image 스크린샷 2025-02-05 오후 6 18 23

@minjeongss
Copy link
Member Author

minjeongss commented Feb 5, 2025

호스트 캐릭터 위치를 우측으로 변경했어요!
피그마에서 ref가 아닌 frame desing 2.0엔 3R가 blue가 아닌 brown이어서, 이 친구도 추가했습니다! 😆

@aaahyesu

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: 참여 가능한 방 화면 구현
2 participants