Skip to content

Conversation

@crazyupinc-design
Copy link
Collaborator

@crazyupinc-design crazyupinc-design commented Dec 15, 2024

이슈 번호

close #19

변경 사항 요약

인풋필드 Props Type

interface InputFieldProps {
  value: string;
  onChange: (e: React.ChangeEvent<HTMLInputElement>) => void;
  type?: 'text' | 'email' | 'password' | 'name' | 'passwordConfirm';
  placeholder?: string;
  label?: string;
  compareValue?: string;
}

사용예시

 {/* 생일 필드 */}
      <InputField
        value={birthday}
        onChange={handleBirthdayChange}
        type="text"
        label="생일"
        placeholder="생일을 선택하세요"
        layout="horizontal"
      />
 {/* 이메일 필드 */}
      <InputField
        value={email}
        onChange={handleEmailChange}
        type="email"
        label="이메일"
        placeholder="이메일을 입력하세요"
        layout="vertical"
      />

signup페이지는 미완성이므로 리뷰하지 않으셔도 됩니다.

테스트 결과

image

Copy link
Contributor

@junghwaYang junghwaYang left a comment

Choose a reason for hiding this comment

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

생일 input에서는 달력이 노출되게끔 사용하신다하셨는데 해당 부분에 대한 코드는 없는것같아요!

@junghwaYang
Copy link
Contributor

junghwaYang commented Dec 15, 2024

dev 최신사항 변경해놨고 merge 컴플릿 나는거 해결해서 푸시했어요~
그리고 다음부터는 하나의 브랜치의 한개의 작업만 부탁드려요!

페이지에 테스트 하고 브랜치를 옮겨서 작업 할 경우 git stash 하고 브랜치 옮기신 후에 올바른 브랜치에서 커밋올리시면 됩니당

Copy link
Collaborator

@haksoo0918 haksoo0918 left a comment

Choose a reason for hiding this comment

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

이 컴포넌트가 유지되는 건가요? 제가 정확히는 잘 몰라서...
제가 프로필 필드하고 헷갈린거 같아요. 💦
다시 리뷰 필요하시면 리뷰어 제 아이디 옆에 're-request review' 버튼 눌러주세요. 😄

인풋필드에 가로모드 추가
@junghwaYang
Copy link
Contributor

생일 input에서는 달력이 노출되게끔 사용하신다하셨는데 해당 부분에 대한 코드는 없는것같아요!

이거도 같이 작업끝난거죠?? 커밋에 같이 올라와있더라구요!

@crazyupinc-design
Copy link
Collaborator Author

생일 input에서는 달력이 노출되게끔 사용하신다하셨는데 해당 부분에 대한 코드는 없는것같아요!

생일 레이블일때 인풋창 클릭시 데이픽커(달력)나오도록 구현했습니다
데이픽커는 추후 추가개발때 스타일 수정하겠습니다

@crazyupinc-design
Copy link
Collaborator Author

생일 input에서는 달력이 노출되게끔 사용하신다하셨는데 해당 부분에 대한 코드는 없는것같아요!

이거도 같이 작업끝난거죠?? 커밋에 같이 올라와있더라구요!

생일 레이블일때 인풋창 클릭시 데이픽커(달력)나오도록 구현했습니다
데이픽커는 추후 추가개발때 스타일 수정하겠습니다

Copy link
Collaborator

@haksoo0918 haksoo0918 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다. 👍

@crazyupinc-design crazyupinc-design merged commit 05029fd into develop Dec 16, 2024
1 check passed
@crazyupinc-design crazyupinc-design deleted the feature/#19_인풋필드-컴포넌트 branch December 16, 2024 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📁 컴포넌트 컴포넌트 개발 관련 작업

Projects

Status: 완료

Development

Successfully merging this pull request may close these issues.

인풋필드 컴포넌트

4 participants