Portfolio

코드 리뷰 시에 고쳐도 되지 않을 것 같은 건 바로 approve를 누르자

[DK-287] 개인 프로필 UI 구성 by kyubumjang · Pull Request #25 · prgrms-web-devcourse/Team_04_SFam_FE

📌 개요 개인 프로필 UI 구성했습니다! 👩‍💻 구현 내용 styles 폴더 common.ts에 제가 사용하는 Container, Wrapper, Text 스타일들을 넣어놨으니 나중에 UI 구성할때 사용하면 좋을 것 같아요! ✅ PR 포인트 & 궁금한 점 interface 폴더 만들어서 API 때 사용할 타입이나 Style에 사용하는 타입...

https://github.com/prgrms-web-devcourse/Team_04_SFam_FE/pull/25

리뷰

interface 폴더 ⇒ types 폴더

⇒ rebase시 types로 합치기

로그아웃

버튼 ⇒ 상단과 동일하게 span

⇒ 개인 프로필 수정

Divider 컴포넌트와 Header의 하단 테두리가 길이만 다른데

똑같이 가는 것이 좋을 것 같다

⇒ 리팩터링 때 삭제 or 색깔을 바꾸자

[DK-290] 메인 페이지 UI 구성 by dustmddus · Pull Request #26 · prgrms-web-devcourse/Team_04_SFam_FE

Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes.

https://github.com/prgrms-web-devcourse/Team_04_SFam_FE/pull/26

리뷰

공고가 잘린다

제목, 뱃지, 날짜가 일렬인데 모바일 특성 상 세로 배치가 좋을 것 같다

제목과 뱃지, 날짜를 한 세트로 구분하면 되지 않을까

테마 이용 관련해서


import theme from '@styles/theme'

위 임포트문은 사용하지 않기로 했던 것 같은데 심지어 어떤 곳은 쓰고 어떤 곳은 안 쓰고..


${(props)=> props.theme.color.gray100}

테마 주입을 쓰는 의미가 사라진다
이렇게 하고 싶었다면 ThemeProvider를 없애는 쪽이 좋지 않을까

네비게이터 아이콘 크기를 줄어듬

파일명 : style.ts ⇒ styles.ts

린트 룰은 주석처리하지 않기로..

임시로 해둔 것 아니면 삭제 후 타입 처리

컨테이너로 감싸져 있는데 페이지에서 div 로 한번 더 감쌀 필요가 있을까

[DK-294] 팀 프로필 UI 구성 by kyubumjang · Pull Request #27 · prgrms-web-devcourse/Team_04_SFam_FE

https://github.com/prgrms-web-devcourse/Team_04_SFam_FE/pull/27

리뷰

Heading 컴포넌트에서 :id vs [id] ⇒ 후자로 해야 동작!

object 형태 의 스타일드 컴포넌트 고민해볼 필요가 있음

코드 훨씬 예쁜거 같으니까 리팩터링 때 만들어진 것은 삭제

더미 데이터를 만드는 경우 파일 분리를 하면 좋을 것 같다

API 연동 작업 시에 사라질 것들인데 변경 사항 확인이 쉽다

⇒ 파일 삭제 vs 같은 파일 변경 사항 비교

구현 시 속도 때문에 파일 분리 하는쪽으로

[DK-288] 공고 상세 페이지 UI 구현 by dustmddus · Pull Request #29 · prgrms-web-devcourse/Team_04_SFam_FE

https://github.com/prgrms-web-devcourse/Team_04_SFam_FE/pull/29

리뷰

PR 문항 안 쓰는 거 삭제

PR 타이틀 형식 맞추자

[Dk 290] ⇒ [DK-290]

게시글 URL ⇒ post vs match

API와 맞출 필요가 있는지 고민이 필요하다

post/create ⇒ post

filter map 메서드의 콜백 함수 매개변수로 i 는 좋지 않다고 생각한다

ex. posts 배열이면 posts.map(post)

안 쓰는 코드는 버리는게 좋을 것 같다

아래 같은 경우 matchType에 따른 각 뱃지를 만들어 주는 것이 좋을 것 같다

styled-components에서 matchType을 받아서 분기


<Badge color={matchType === '개인전' ? `${theme.color.primary}` : `${theme.color.yellow}`}>
	{matchType}
</Badge>

[DK-300] 팀 생성 UI 구성 by kyubumjang · Pull Request #31 · prgrms-web-devcourse/Team_04_SFam_FE

https://github.com/prgrms-web-devcourse/Team_04_SFam_FE/pull/31

리뷰

border-radius에 1px solid..

isOpen으로 조건부 렌더링도 하고 isOpen으로 display 컨트롤도 한다.

코드 리뷰