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

Mara-32 나눔 목록 조회 화면 구현 #8

Merged
merged 25 commits into from
Jan 30, 2024
Merged

Mara-32 나눔 목록 조회 화면 구현 #8

merged 25 commits into from
Jan 30, 2024

Conversation

hyeseon-han
Copy link
Member

요구사항

이슈번호: Mara-32

작업내용

  • 나눔 목록 조회 화면 구현
  • 아이콘 컴포넌트 assets 내부 개별 생성, barrel파일 사용
  • 기존 아이콘 사용 부분 마이그레이션 , 기존 아이콘 컴포넌트 파일 제거

테스트 결과 또는 방법

  • 피그마 화면 요건 확인
  • 아이콘 적용 확인

@hyeseon-han
style: pc 화면시 영역 밖 배경색 변경
1fb20dc
@hyeseon-han
chore: Angle Icon 추가
483e1bb
@hyeseon-han
fix: Header component backgroundColor props 추가
6f818d3
@hyeseon-han
style: global.css body color 설정
8510296
@hyeseon-han
chore: dayjs install
6de54aa
@hyeseon-han
feat: TabButton component 생성
09c48fe
@hyeseon-han
feat: 나눔 리스트 아이템 컴포넌트 (ShareListItem) 생성
75adbb7
@hyeseon-han
feat: 나눔 탭 화면 구현
af0dd03
@hyeseon-han
chore: material-tailwind/react insall
37406b6
@hyeseon-han
chore: dayjs locale 설정
0c2593f
@hyeseon-han
chore: chakra-ui/react, emotion/react, emotion/styled, framer-motion …
e20c886 
…install
@hyeseon-han
feat: Radio component 생성
04890f5
@hyeseon-han
feat: 나눔 탭 화면 바텀 시트 구현
c262661
@hyeseon-han
chore: globals.css font-family 적용 수정
5d574fc
@hyeseon-han
chore: 사용하지 않는 svg파일 제거
c87b772
@hyeseon-han
chore: assets directory 구조 변경, 홈에 사용되는 svg파일 위치이동
962efcd
@hyeseon-han
chore: lint 수정
b5cb324
@hyeseon-han
fix: 사용되고 있던 svg파일 컴포넌트화, 적용
c24435a
@hyeseon-han
fix: 기존 Icon 파일제거, 내부 Icon svg assets으로 이동, barrel 파일 추가
3a433af
a-honey
a-honey reviewed Jan 28, 2024
View reviewed changes
Copy link
Member

@a-honey a-honey 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.

아이콘 확인했습니다
수정 감사합니다 🌸

@a-honey a-honey self-requested a review January 28, 2024 09:49
a-honey
a-honey approved these changes Jan 28, 2024
View reviewed changes
Copy link
Member

@a-honey a-honey left a comment

Choose a reason for hiding this comment

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

승인을 안해서 승인했습니다!

choipureum
choipureum approved these changes Jan 29, 2024
View reviewed changes
Copy link
Member

@choipureum choipureum left a comment

Choose a reason for hiding this comment

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

ICON 및 여러 부분 리팩토링 감사드립니다. 깰끔하네요..!
너무너무 고생하신게 보이네요🥲
몇 개 코멘트 남겼으니 확인 부탁드리고 고생 많으셨습니다!🙇‍♂️

@choipureum
Copy link
Member

LGTM👍

@hyeseon-han
Copy link
Member Author

@choipureum 양이 많았는데도 꼼꼼하게 피드백 주셔서 감사합니다 🙇‍♀️
수정이 더 필요한 부분은 리서치 후에 일괄 수정하도록 하겠습니다 ☺️

@hyeseon-han hyeseon-han merged commit 387aca7 into main Jan 30, 2024
1 check passed
@hyeseon-han hyeseon-han deleted the MARA-32 branch January 30, 2024 00:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@a-honey a-honey a-honey approved these changes

@choipureum choipureum choipureum approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@hyeseon-han @choipureum @a-honey