Skip to content

Conversation

@cozy-ito
Copy link
Collaborator

@cozy-ito cozy-ito commented Apr 24, 2025

#️⃣연관된 이슈

ex) #이슈번호, #이슈번호

Closes #14

📝 PR 유형

해당하는 유형에 'x'로 체크해주세요.

  • 기능 추가 (Feature)
  • 버그 수정 (Bug Fix)
  • 코드 개선 (Refactoring)
  • 스타일 변경 (UI/UX)
  • 문서 작업 (Documentation)
  • 환경 설정 (Configuration)
  • 기타 (Other)

📝작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

공통 컴포넌트 중 TextField 컴포넌트를 구현합니다.
자세한 설명은 TextField Wiki 페이지를 참고해주세요.

스크린샷 (선택)

💬리뷰 요구사항(선택)

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

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

  • as props를 통해 input 태그와 textarea 태그를 호환되게 사용하려다보니,
    forwardRef를 추가하는 과정에서 타입 정의 코드가 꽤 복잡해진 것 같습니다 ,
    아예 분리해서 작성하는게 나을 지 고민입니다 하하 😅

@cozy-ito cozy-ito added the Feature (기능 추가) 기능을 추가합니다. label Apr 24, 2025
@netlify
Copy link

netlify bot commented Apr 24, 2025

Deploy Preview for thejulge1 ready!

Name Link
🔨 Latest commit 5f041cd
🔍 Latest deploy log https://app.netlify.com/sites/thejulge1/deploys/680df84d706ffc0008e53455
😎 Deploy Preview https://deploy-preview-27--thejulge1.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Collaborator

@ghdtnals ghdtnals left a comment

Choose a reason for hiding this comment

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

수고하셨습니다👍👍

@ghdtnals ghdtnals self-requested a review April 25, 2025 09:42
@ghdtnals
Copy link
Collaborator

ghdtnals commented Apr 25, 2025

input과 textarea의 통합으로 forwardRef 쪽 타입이 조금 복잡해진 것 같긴 하네요..!
TextField.input, TextField.Textarea처럼 명시적으로 분리해도 괜찮을 것 같긴 합니다!🙂

Copy link
Collaborator

@almighty55555 almighty55555 left a comment

Choose a reason for hiding this comment

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

수민님과 같은 의견입니다! TextField.input, TextField.Textarea로 분리하는 게 좋을 것 같아요

@cozy-ito cozy-ito merged commit 957aa8b into dev Apr 27, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature (기능 추가) 기능을 추가합니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

공통 컴포넌트 구현 (Input)

4 participants