[Review]

[lang92]

리뷰 PR입니다. 머지하시면 안되고, 각자 작업하신 내용에 대한 리뷰를 반영해 주세요!

[lang92]

리뷰는 반복해서 달아드리지 않았으니, 본인이 작업한 내용이 아니라도 모두 읽어 주세요!
그리고 나서 각자 작업한 내용에 반영할 내용 반영해 주시면 됩니다.

또 제 개인적인 기준에서는 전체적으로 컴포넌트를 너무 잘게 쪼개셨는데, 이 부분은 각자 작업해 보시면서 느끼신 불편함이나 이점 등에 대해 잘 생각해 보시면 좋을 것 같습니다.
그리고 src 폴더 하위의 pages 폴더와 components 폴더 각각의 역할에 대해 설명드리면,

• pages: 페이지 컴포넌트
• components: 여러 페이지에서 공용으로 사용되는 컴포넌트

인데, 현재 components 폴더에 모든 하위 컴포넌트들이 들어가 있네요. 이 부분도 수정해 주세요!

[lang92]

Suggested change

	{icon?.length > 0 && (
	<img src={process.env.PUBLIC_URL + `/${icon}.png`} alt={`${icon}`} />
	)}
	{icon && (
	<img src={`${process.env.PUBLIC_URL}/${icon}.png`} alt={`${icon}`} />
	)}

[lang92]

• 함수 컴포넌트의 선언 방식도 통일하면 좋을 것 같습니다.
• props는 매개변수 자리에서도 받아올 수 있습니다.

Suggested change

	function Input(props) {
	const { type, className, name, placeholder } = props;
	const Input = ({ type, className, name, placeholder }) => {

[lang92]

Input 컴포넌트에서 최상위 요소인 label에 className 부여하고 nesting해주세요! 그래야 다른 요소에 영향을 미치지 않을 것 같습니다.

[lang92]

선택자 간에는 한 줄씩 띄워서 가독성을 높여 주세요!

Suggested change

	.tag {
	position: absolute;
	left: 20px;
	top: 30px;
	span {
	padding: 5px 10px;
	.tag {
	position: absolute;
	left: 20px;
	top: 30px;
	span {
	padding: 5px 10px;