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

[Refactor] Input 컴포넌트 리팩토링 #362

Merged
merged 2 commits into from
Dec 7, 2023

Conversation

ttaerrim
Copy link
Member

@ttaerrim ttaerrim commented Dec 7, 2023

…하도록 리팩토링

설명

  • [REFACTOR] 컴포넌트 리팩토링 #361
  • Input 컴포넌트 사용 시, Input 컴포넌트의 타입으로 명시하지 않은 타입을 추가하여 사용하고 싶다면 일일이 Input 컴포넌트에 추가해야 하는 번거로움이 발생. 이를 InputHTMLAttributes 타입과 ...rest props를 사용하여 해결.

완료한 기능 명세

  • Input 컴포넌트 리팩토링

스크린샷

기능 작업에 대한 스크린샷/화면 녹화 있을 경우 첨부하기

리뷰 요청 사항

특별히 리뷰해 주었으면 하는 부분, 고민되는 부분 기재하기

@ttaerrim ttaerrim added ✨ enhancement 개선 또는 피드백 반영 🎨 Front-end labels Dec 7, 2023
@ttaerrim ttaerrim self-assigned this Dec 7, 2023
@ttaerrim ttaerrim marked this pull request as ready for review December 7, 2023 05:30
list,
}: InputProps) {
export function Input(props: InputProps) {
const { label = '', errorMessage = '', ...rest } = props;
Copy link
Collaborator

Choose a reason for hiding this comment

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

이런 방법으로 기본값을 지정할 수도 있군요 👀

@ttaerrim ttaerrim merged commit 20ee8ac into boostcampwm2023:develop Dec 7, 2023
1 check passed
@ttaerrim ttaerrim deleted the 361-input branch December 7, 2023 06:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ enhancement 개선 또는 피드백 반영 🎨 Front-end
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants