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

feat(input): Input 컴포넌트 구현 #23

Open
wants to merge 22 commits into
base: main
Choose a base branch
from
Open

feat(input): Input 컴포넌트 구현 #23

wants to merge 22 commits into from

Conversation

froggy1014
Copy link

변경사항

Compound Pattern으로 구현해보려고 했었는데,

    <Field.Root>
      <Field.Label>Label</Field.Label>
      <Field.Input />
      <Field.ErrorText>Error Info</Field.ErrorText>
    </Field.Root>

저희 Input 관련해서는 타입마다 다 컴포넌트로 나눠져 있는거 같습니다.

<InputNumber /> 
<Radio /> 
<CheckBox /> 
<Input />

각 컴포넌트는 분리하고, <Form /> 같은 컴포넌트로 감싸서 그 안에서 유틸리티 컴포넌트를 제공하는 방향이 더 낫겠다 싶어서, Input은 딱 그 역할만 잘할 수 있게끔 작성을 해보았습니다.

시각자료

default disabled withActionButton
image image image

체크리스트

  • 기능 명세를 작성하였나요?
  • 테스트 코드를 작성하였나요?

추가 논의사항

임시로 spacing, color를 정한 부분이 있는데 그건 추후 컨벤션 정해서 고쳐나가야할 것 같습니다..

그 외 질문이나.. 관련해서 코멘트 달아주시면 감사하겠습니다 -

@froggy1014 froggy1014 self-assigned this Nov 17, 2024
@froggy1014 froggy1014 linked an issue Nov 17, 2024 that may be closed by this pull request
@noahluftyang
Copy link
Collaborator

코드 포맷팅이 제대로 적용되어있지 않은 것 같은데, vscode에 biome extension 설치 되어있는지 확인 부탁드려도 될까요?!

packages/Input/src/Input.tsx Outdated Show resolved Hide resolved
packages/Input/src/Input.tsx Outdated Show resolved Hide resolved
packages/Input/vitest.setup.ts Outdated Show resolved Hide resolved
packages/Input/src/Input.test.tsx Outdated Show resolved Hide resolved
packages/Input/src/Input.test.tsx Outdated Show resolved Hide resolved
packages/Input/src/Input.tsx Outdated Show resolved Hide resolved
@noahluftyang
Copy link
Collaborator

그래도 전체적으로 코드가 잘 정돈되어있고 고민의 흔적들이 많이 보여서 좋았어요!! 👍👍👍

@froggy1014
Copy link
Author

froggy1014 commented Nov 18, 2024

코드 포맷팅이 제대로 적용되어있지 않은 것 같은데, vscode에 biome extension 설치 되어있는지 확인 부탁드려도 될까요?!

맞습니다. 어째서인지 Disable이 되어있었네요. 어째...

@noahluftyang

피드백 반영했습니다. 감사합니다 ㅎㅎ

Copy link
Collaborator

@noahluftyang noahluftyang left a comment

Choose a reason for hiding this comment

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

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Input
2 participants