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: 퀴즈 결과 페이지 추가 #76

Merged
merged 8 commits into from
Sep 2, 2024
Merged

feat: 퀴즈 결과 페이지 추가 #76

merged 8 commits into from
Sep 2, 2024

Conversation

azure-553
Copy link
Collaborator

@azure-553 azure-553 commented Sep 2, 2024

#️⃣ 이슈 번호

ex) - #이슈번호


📝 작업 내용

  • 결과페이지 레이아웃을 제작했습니다

📸 스크린샷

Screenshot 2024-09-02 at 10 42 58 PM

💬 리뷰 요구사항/참고 사항

  • Mockup 데이터를 활용했기 때문에 스크린샷을 위와 같이 담았지만, 데이터가 바뀌면 이미지 및 텍스트가 변경됩니다
  • 퀴즈 결과에 대한 svg파일 인덱스를 quiz_result_00.svg ~ quiz_result_03.svg으로 설정했습니다. (정답 개수가 0개인 경우와 맞추기 위해 파일명을 조정했습니다.
  • cross 버튼 클릭시 바텀시트 나타나는 부분은 추구 적용할 예정입니다

Copy link

github-actions bot commented Sep 2, 2024

Copy link
Collaborator

Choose a reason for hiding this comment

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

quiz_result_02.svg랑 같은 파일이 들어가있어요~

Copy link
Collaborator

Choose a reason for hiding this comment

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

resultText는 constants에서 관리하고 imageSrc, altText는 getQuizResult 함수 내에서 구현하는 걸로 분리한 이유가 있나요?

imgSrc, altText도 constants 폴더 resultData에서 같이 상수화해서 관리하는 게 일관성 있을 것 같아요
splashData에서도 이미 그렇게 구현되어 있듯이..?

Copy link

github-actions bot commented Sep 2, 2024

@lsy20140
Copy link
Collaborator

lsy20140 commented Sep 2, 2024

image
image

파일 export 하는 방식에서 일관성이 떨어지는 거 같아요
기능 구현에서 중요한 부분은 아니어도 시간 나면 꼭 통일해서 수정해주세요

- 파일 export 방식 변경 및 img, altText를 resultData로 관리하기
Copy link

github-actions bot commented Sep 2, 2024

@azure-553 azure-553 merged commit 0f9c128 into main Sep 2, 2024
2 checks passed
@azure-553 azure-553 self-assigned this Sep 9, 2024
@azure-553 azure-553 added ✨ Feature 기능 개발 🎨 Html&css 마크업 & 스타일링 labels Sep 9, 2024
@azure-553 azure-553 linked an issue Sep 9, 2024 that may be closed by this pull request
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발 🎨 Html&css 마크업 & 스타일링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: 퀴즈 결과 페이지 추가
2 participants