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.

Sign up for GitHub

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

Jump to bottom

일부 컴포넌트 구현 및 스토리북 추가 #16

Merged
merged 43 commits into from
Aug 5, 2024
Merged

일부 컴포넌트 구현 및 스토리북 추가 #16

merged 43 commits into from
Aug 5, 2024

Conversation

froggy1014
Copy link
Collaborator

PR 타입 ( 하나 이상의 PR 타입을 선택해주세요 )

  • 기능 추가
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트
  • 기타 사소한 수정

개요

일단 먼저 나와있는 컴포넌트들 조금 작업해놨습니다.

변경 사항

대부분 컴포넌트 구현 작업이라 �스토리북 보시면 될 것 같습니다.

@froggy1014
🔨 settings: svgr/webpack 설치 #13
c9d9ef4
@froggy1014
🔨 settings: svg setting in nextjs.config #13
4ae869b
@froggy1014
🔨 settings: svgr 글로벌 타입 선언 #13
ac2f9af
@froggy1014
💅 style: svg 파일 fill="currentColor" 설정 및 re-export 파일 생성 #13
333d2cd
@froggy1014
Merge branch 'develop' into style/icon-update
64ec6fb
@froggy1014
🔧 chore: .keep 삭제
a676927
@froggy1014
💅 style: add dummy stories for chromatic build
ce50437
@froggy1014
💅 style: squareTabButton 컴포넌트 UI 구축
4849bcb
@froggy1014
💅 style: 예제 사진 추가 #15
2dbf638
@froggy1014
🔨 settings: radix/theme 삭제, tailwindcss/lineclamp 설치 #15
7a55918
@froggy1014
🔧 chore: radix theme 관련 코드 삭제 #15
2d4cd24
@froggy1014
🔨 settings: 스토리북에도 SVGR 설정 추가 #15
698d07b
@froggy1014
🔧 chore: 단순 포매팅
dfc42fc
@froggy1014
🔧 chore: radix theme 설정 삭제
f1183d7
@froggy1014
💅 style: 버튼 컴포넌트 추가 #15
1ad48d5
@froggy1014
🔨 settings: 폰트 파일 위치 변경 및 스토리북에 적용 완료 #15
72225eb
@froggy1014
✨ feat: 아이콘 버튼 추가 #15
f6da6bc
@froggy1014
✨ feat: chip 컴포넌트 구현 및 스토리북 추가 #15
504030c
@froggy1014
✨ feat: tag 컴포넌트 구현 및 스토리북 추가
5ad0ad1
@froggy1014
✨ feat: list 컴포넌트 구현 및 스토리북 추가 #15
c1e1f28
@froggy1014
🔧 chore: iconButton active 필드 옵셔널로 변경
e33fed3
@froggy1014
💅 style: typography lintheight 1.5 -\> 1.2로 수정 #15
3af387f
@froggy1014
✨ feat: input 컴포넌트 구현 및 스토리북 추가 #15
0442352
@froggy1014
✨ feat: header 컴포넌트 구현 및 스토리북 추가 #15
0920ac6
@froggy1014
📝 docs: 코어 스토리북 컴포넌트을 Core 대분류안으로 분류 #15
230b068
@froggy1014
🔧 chore: directory rename common -\> core
4907eb8
@froggy1014
🔨 settings: 스토리북 빌드 파일 ignore 처리
f3e9206
@froggy1014
🔧 chore: 스크린 사이즈 네이밍 변경
b5fea68
@froggy1014
💅 style: 헤더, 바디, 네비게이션바를 선언적으로 구조설정하기 위해 view 파일 생성 #15
54c4d38
@froggy1014
✨ feat: rating 컴포넌트 구현 #15
989c7cd
@froggy1014 froggy1014 self-assigned this Aug 4, 2024
@vercel Vercel
Copy link

vercel bot commented Aug 4, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
fiesta ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 5, 2024 4:03pm

@froggy1014 froggy1014 linked an issue Aug 4, 2024 that may be closed by this pull request
컴포넌트 구현 및 스토리북 정리 #15
Open
@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 4, 2024

💅 ## storybook-URL - https://66a939e10fecfdf69816c066-glrvgrhexl.chromatic.com/

punchdrunkard
punchdrunkard approved these changes Aug 5, 2024
View reviewed changes
Copy link
Member

@punchdrunkard punchdrunkard left a comment •
edited
Loading

Choose a reason for hiding this comment

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

엄청난 구현양이시군요...... 수고많으셨습니다👍
리뷰는 항상 그렇듯이 개인적인 의견일 뿐이니 가볍게 읽어주세요!
감사합니다.

+) 저번 PR 부터 CI의 UI Test가 제대로 수행되지 않는 것 같습니다! pending 에서 진행이 안되는 것 처럼 보이네요.
image

image

src/components/core/Button/IconButton/IconButton.tsx
export interface Props
extends Omit<HtmlHTMLAttributes<HTMLButtonElement>, "children"> {
active?: boolean;
icon: React.ReactNode;
Copy link
Member

Choose a reason for hiding this comment

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

중요X : https://tech.socarcorp.kr/dev/2022/09/06/react-icon-component.html
위와 같은 방식으로 Icon 을 관리하는 사례도 있더라고요! 조금 예전 자료긴하지만 공유드립니다😀

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

음 일단 이글 말미에도 SVGR이 마지막에 언급되는 부분을 보아.. 일단 SVGR 형태로 유지하도록 하겠습니다.

src/components/core/Button/RegisterButton/RegisterButton.tsx Outdated Show resolved Hide resolved
src/components/core/Chip/BasicChip/BasicChip.tsx Outdated Show resolved Hide resolved
src/components/core/Input/SearchInput.tsx Outdated Show resolved Hide resolved
src/components/core/List/FestivalTile.stories.tsx Outdated Show resolved Hide resolved
src/components/core/Button/BaisButton/BasicButton.tsx Outdated Show resolved Hide resolved
@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 5, 2024

💅 ## storybook-URL - https://66a939e10fecfdf69816c066-totqsxnjtw.chromatic.com/

@froggy1014
Copy link
Collaborator Author

엄청난 구현양이시군요...... 수고많으셨습니다👍 리뷰는 항상 그렇듯이 개인적인 의견일 뿐이니 가볍게 읽어주세요! 감사합니다.

+) 저번 PR 부터 CI의 UI Test가 제대로 수행되지 않는 것 같습니다! pending 에서 진행이 안되는 것 처럼 보이네요. image

image

그리고 UI Tests는 보니까 Chromatic에서 디자이너 분들이 Component 보고 Reviews를 다 해줄때까지 Pending인거같아요 ㅎㅎ..

@froggy1014 froggy1014 merged commit 3147c7a into develop Aug 5, 2024
5 of 6 checks passed
@froggy1014 froggy1014 deleted the feature/onboarding-components branch August 5, 2024 16:06
@froggy1014 froggy1014 mentioned this pull request Aug 5, 2024
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@punchdrunkard punchdrunkard punchdrunkard approved these changes

Assignees

@froggy1014 froggy1014

Labels
style
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

컴포넌트 구현 및 스토리북 정리
2 participants
@froggy1014 @punchdrunkard