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

[FE] design: 캐러셀을 정가운데로 위치시킨 뒤, 자동 슬라이드 전환 시간을 6초로 설정한다. 그리고 전반적인 디자인을 수정한다. #465

Merged
merged 6 commits into from
Aug 20, 2024

Conversation

soosoo22
Copy link
Contributor

@soosoo22 soosoo22 commented Aug 20, 2024


🚀 어떤 기능을 구현했나요 ?

  • ReviewZonePage 및 홈 페이지 캐러셀을 정가운데로 위치시켰습니다.
  • 자동 슬라이드 전환 시간을 6초로 변경했습니다.
  • 홈 페이지 폼 영역에 모든 placeholder를 제거했습니다.

🔥 어떻게 해결했나요 ?

  • 정가운데에 와야하는 컴포넌트 같은 경우 align-items: center;을 추가했습니다. 그리고 슬라이드 전환 시간은 setTimeout 을 6초로 변경했습니다.
  • Input 공동 컴포넌트 propsplaceholder가 옵셔널로 되어 있어서 사용처에서 해당 속성을 제거했습니다.

📝 어떤 부분에 집중해서 리뷰해야 할까요?

📚 참고 자료, 할 말

  • 성장하는 괴물! 커비! 화이팅!!😘

Copy link
Contributor

@chysis chysis left a comment

Choose a reason for hiding this comment

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

고생했어요~

type="text"
placeholder="이름"
/>
<Input id="reviewee-name" value={revieweeName} onChange={handleNameInputChange} type="text" />
Copy link
Contributor

Choose a reason for hiding this comment

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

reviewee-name 이라는 아이디가 프로젝트 기능이 커지면 동일한 id가 나올 수 있을 것 같아요.

id를 사용한다면, 유일한 아이디 값이 될 수 있도록 react의 useId를 사용하는 것은 어떨까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

useId라는 훅이 있었군요! 바다 덕분에 하나 알아갑니다!!

  const INPUT_ID = {
    reviewName: `review-name-${useInputId}`,
    projectName: `project-name-${useInputId}`,
    password: `password-${useInputId}`,
  };
        
  <S.Label htmlFor={INPUT_ID.reviewName}>본인의 이름을 적어주세요</S.Label>
  <Input id={INPUT_ID.reviewName} value={revieweeName} onChange={handleNameInputChange} type="text" />

Copy link
Contributor

@ImxYJL ImxYJL left a comment

Choose a reason for hiding this comment

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

디자인 작업 수고 많았어요~

Copy link
Contributor

@BadaHertz52 BadaHertz52 left a comment

Choose a reason for hiding this comment

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

🚀🚀🚀🚀🚀

@BadaHertz52 BadaHertz52 merged commit 2d28ba7 into develop Aug 20, 2024
4 checks passed
@donghoony donghoony deleted the fe/design/462-updated-design branch August 23, 2024 02:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
4 participants