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: 퀴즈 결과 카드 컴포넌트 추가 및 스토리북 작성 #56

Merged
merged 14 commits into from
Aug 27, 2024

Conversation

azure-553
Copy link
Collaborator

#️⃣ 이슈 번호

ex) - #이슈번호


📝 작업 내용

퀴즈 결과 부분에 쓰일 결과 카드의 컴포넌트 스토리북을 작성하였습니다.


📸 스크린샷

Screenshot 2024-08-27 at 6 51 11 PM

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

처음 스토리북을 사용해서 미숙한 부분이 많지만, 잘못된 부분이나 개선했으면 하는 부분 이야기해주시면 반영하도록 하겠습니다!

Copy link

@azure-553 azure-553 changed the title Feat/#55 feat: 퀴즈 결과 카드 컴포넌트 추가 및 스토리북 작성 Aug 27, 2024
@azure-553 azure-553 added 🎨 Html&css 마크업 & 스타일링 📃 Docs 문서 작성 및 수정 (README.md 등) labels Aug 27, 2024
Comment on lines 20 to 21
isCorrect={true}
isMarked={true}
Copy link
Member

Choose a reason for hiding this comment

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

P3

={true} 형태는 축약 가능해요~

 <QuizReultCard
        wordId={1}
        isCorrect
        isMarked

Comment on lines 22 to 28
name={'아젠다(Agenda)'}
selectedOptionDescription={
'완수해야 하는 실무 내용을 사전에 정리해 둔 항목들'
}
answerOptionDescription={
'완수해야 하는 실무 내용을 사전에 정리해 둔 항목들'
}
Copy link
Member

Choose a reason for hiding this comment

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

P3

단순 string이라면 중괄호 생략 가능할 거예요

Comment on lines 21 to 34
{isCorrect ? (
<Image
src={'/icons/correct.svg'}
alt={'correct.svg'}
width={20}
height={20}
/>
) : (
<Image
src={'/icons/wrong.svg'}
alt={'wrong.svg'}
width={20}
height={20}
/>
Copy link
Member

Choose a reason for hiding this comment

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

P3

지금의 컴포넌트는 실시간으로 isCorrect 값이 변경되진 않으므로 렌더링 성능 이슈가 없겠지만
보편적으로 조금 더 클린한 코드는 아래에 가깝다고 생각해요!

<Image
  src={isCorrect ? '/icons/correct.svg' : '/icons/wrong.svg'}
  alt={isCorrect ? 'correct.svg' : 'wrong.svg'}
  width={20}
  height={20}
/>

</div>
<BookmarkButton wordId={wordId} isMarked={isMarked} />
</div>
{isCorrect ? null : (
Copy link
Member

Choose a reason for hiding this comment

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

P3

isCorrect && (
이렇게 표현하는 것이 더 좋겠네요~

Copy link
Member

Choose a reason for hiding this comment

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

P4

파일명 카멜케이스로 디테일 챙겨볼까요!

import { ExplanationInfo } from '@/types/quizresult'
import BookmarkButton from '../BookmarkButton'

export default function QuizReultCard({
Copy link
Collaborator

Choose a reason for hiding this comment

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

컴포넌트 이름 오타 났어요!~

@lsy20140
Copy link
Collaborator

lsy20140 commented Aug 27, 2024

image

스토리북상으로 피그마랑 비교했을 때 텍스트 색상, 굵기, 색상, 간격 다 제대로 적용이 안 된 것 같아요 수정해주세욥

단순한 문자열 중괄호 생략
Copy link

QuizReultCard -> QuizResultCard
Copy link

Copy link

조건부 렌더링에 대한 성능 고려
Copy link

@azure-553
Copy link
Collaborator Author

azure-553 commented Aug 27, 2024

image

스토리북상으로 피그마랑 비교했을 때 텍스트 색상, 굵기, 색상, 간격 다 제대로 적용이 안 된 것 같아요 수정해주세욥

엇..!! 컴포넌트 크기는 화면에 따라 그때 그때 조절하는 줄 알았는데 아니였군요! 바로 적용하도록 하겠습니다!
text 색상 및 굵기에 대한 부분은 확인했는데, 혹시 다른 컴포넌트 내 색상 중에 제대로 적용되지 않은 부분이 어디일까용..? @lsy20140

@lsy20140
Copy link
Collaborator

lsy20140 commented Aug 27, 2024

image
스토리북상으로 피그마랑 비교했을 때 텍스트 색상, 굵기, 색상, 간격 다 제대로 적용이 안 된 것 같아요 수정해주세욥

엇..!! 컴포넌트 크기는 화면에 따라 그때 그때 조절하는 줄 알았는데 아니였군요! 바로 적용하도록 하겠습니다! text 색상 및 굵기에 대한 부분은 확인했는데, 혹시 다른 컴포넌트 내 색상 중에 제대로 적용되지 않은 부분이 어디일까용..? @lsy20140

아 제가 중복으로 잘못 썼네요 그냥 텍스트 색상 말한 거였어요!
그리고 컴포넌트 크기에 대해서 말한 게 아니라 간격 말한거예요!

@azure-553
Copy link
Collaborator Author

azure-553 commented Aug 27, 2024

image

스토리북상으로 피그마랑 비교했을 때 텍스트 색상, 굵기, 색상, 간격 다 제대로 적용이 안 된 것 같아요 수정해주세욥

image
스토리북상으로 피그마랑 비교했을 때 텍스트 색상, 굵기, 색상, 간격 다 제대로 적용이 안 된 것 같아요 수정해주세욥

엇..!! 컴포넌트 크기는 화면에 따라 그때 그때 조절하는 줄 알았는데 아니였군요! 바로 적용하도록 하겠습니다! text 색상 및 굵기에 대한 부분은 확인했는데, 혹시 다른 컴포넌트 내 색상 중에 제대로 적용되지 않은 부분이 어디일까용..? @lsy20140

아 제가 중복으로 잘못 썼네요 그냥 텍스트 색상 말한 거였어요! 그리고 컴포넌트 크기에 대해서 말한 게 아니라 간격 말한거예요!

앗 네넵! 확인했습니다! 간격 어디 부분인지도 확인했고, 같이 수정해서 올리겠습니다!

Copy link

Copy link

Copy link

Copy link

@azure-553 azure-553 merged commit d78f04b into main Aug 27, 2024
2 checks passed
@azure-553 azure-553 deleted the feat/#55 branch August 27, 2024 16:37
@azure-553 azure-553 linked an issue Aug 27, 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
📃 Docs 문서 작성 및 수정 (README.md 등) 🎨 Html&css 마크업 & 스타일링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: 퀴즈 결과 컴포넌트 스토리북 작성
4 participants