Skip to content

Comments

feat: Dropdown, Tag 컴포넌트 구현#22

Open
danbileee wants to merge 1 commit intoepic/12from
feature/12-dropdown
Open

feat: Dropdown, Tag 컴포넌트 구현#22
danbileee wants to merge 1 commit intoepic/12from
feature/12-dropdown

Conversation

@danbileee
Copy link
Contributor

@danbileee danbileee commented Nov 20, 2021

Issue

Dropdown, Tag 컴포넌트 구현 #12

Changes

  • Dropdown과 Tag 컴포넌트가 시안상 같이 쓰이고 있어서 PR 한번에 작성합니다.
    • Dropdown의 경우 시안에 나온 케이스 테스트만 진행했습니다. single select 등 variation 테스트는 추후 보강하겠습니다.
    • Tag는 시안에 따라 고정 너비이며 텍스트가 길어질 경우 말줄임표 처리했습니다.
  • 컨테이너 외에 하위 컴포넌트에도 className 명시했습니다.

Test Cases

  • 스토리북 UI 테스트

Screen Shots

image

@danbileee danbileee added the feature New feature or request label Nov 20, 2021
@danbileee danbileee requested a review from bbanderson November 20, 2021 11:41
@danbileee danbileee self-assigned this Nov 20, 2021
import { TagProps } from './Tag';

export type { TagProps };
export { default as Tag } from './Tag';
Copy link
Collaborator

Choose a reason for hiding this comment

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

단순히 궁금해서 여쭤봅니다!

예를 들어 아래와 같이 표현하지 않고 type에 대한 import를 위처럼 한단계 더 추가하면 어떤 장점(특징)이 있나요?

export type { TagProps } from './Tag';
export { default as Tag } from './Tag';

@bbanderson
Copy link
Collaborator

작업 감사드립니다! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants