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] 대시보드 뷰 하단 UI 구현 #172

Merged
merged 2 commits into from
Jul 17, 2024

Conversation

jeeminyi
Copy link
Member

작업 내용 🧑‍💻

  • 유기한 브랜치의 새로운 버전 브랜치입니다
  • 유기한 브랜치에 달린 코드리뷰 내용을 반영하였습니다

알게된 점 🚀

기록하며 개발하기!

  • props를 넘겨줄 때 태그를 선언해줄 (맞나요?) 수 있다는 걸 알게 되었습니다 성희의 코드리뷰인데요, 아래에 참고 코드 드립니다. 원리가 궁금해요,, Icon이란느 컴포넌트로 선언해주면 styled를 할 때에는 어떤 태그라고 선언해주면 될까요?
interface TaskSummaryProps {
	text: string;
	data: number;
	unit: string;
	image: string;
	icon: React.FunctionComponent<React.SVGProps<SVGSVGElement>>;
}

function TaskSummary({ text, data, unit, image, icon: Icon }: TaskSummaryProps) {
	return (
		<TaskSummaryLayout image={image}>
			<ProfileImg>
				<Icon />
			</ProfileImg>
			<TextWrapper>
				<SummaryText>{text}</SummaryText>
				<NumberTaskBox>
					<Number>{data}</Number>
					<NumberText>{unit}</NumberText>
				</NumberTaskBox>
			</TextWrapper>
		</TaskSummaryLayout>
	);
}

리뷰 요구사항 💬

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

  • 이전 유기 브랜치 코리는 반영하였습니다! 추가 리뷰사항 있다면 말씀해주시고 따로 없다면 이번에는 그냥 어푸만 해주셔도 됩니닷!

관련 이슈

close #155

스크린샷

스크린샷 2024-07-17 오후 9 39 29

Copy link
Member

@seong-hui seong-hui left a comment

Choose a reason for hiding this comment

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

git 다시 돌려놓으시느라 고생 많으셨습니당 이전에 봤던 pr이고, 이전에 리뷰 남겼던 부분 잘 반영하신 것도 확인했습니다 수고하셨씁니다!!

@jeeminyi jeeminyi merged commit 717825e into develop Jul 17, 2024
2 checks passed
@wrryu09 wrryu09 deleted the feat/#155/summary-component-dashboard branch July 18, 2024 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[FEAT] 대시보드 뷰 하단 컴포넌트 UI 수정
3 participants