theme.jsx 작업완료 (#5)

[kimdohoon2]

작업 내용

• 오픈마인드 피그마에 있는 Foundation을 참고하여 각각의 컬러와 그림자와 테그를 설정하였습니다.

이슈 번호

• 관련 이슈: #05

변경 사항

• X

리뷰 포인트

• 피그마에 있는 컬러값과 폰트값이 일치하는지 봐주시면 감사합니다!
• 그리고 폰트 단위값은 px하는지 혹은 rem으로 하는지 단위값을 정하지 못한거 같아서 리뷰남겨주실때 어떻게 하고싶은지 말씀해주시면 감사합니다.

참고 사항 (Screenshots/References)

• 오픈마인드 피그마 링크 : https://www.figma.com/design/abJyeeWMrJw2YN9wZFHst8/%5BAAA%5D-%E1%84%8B%E1%85%A9%E1%84%91%E1%85%B3%E1%86%AB%E1%84%86%E1%85%A1%E1%84%8B%E1%85%B5%E1%86%AB%E1%84%83%E1%85%B3?node-id=1-796&node-type=frame&t=RzakxJhtqinpbS8C-0

기타 사항 (Additional Context)

• X

[Donghunn-Lee]

테마 스타일 확인했습니다! 이번 건 도훈님의 브랜치로 병합하시는 거라 제가 그냥 머지했습니다!

도훈님 그리고 도훈님이 생성하신 브런치의 하위 브랜치에서 상위로 병합하실 땐 PR없이 로컬에서 독자적으로 feature/1--styles 브랜치로 병합한 후, feature/1--styles의 하위 작업이 모두 완료 됐을 때 develop 브랜치로 PR올리면 되지 않을까요? 제가 이해한 게 맞는지 모르겠네요..!

[kimdohoon2]

음....재가 이해하고 있는건 재가 지금 로컬에서 feature/1--styles브랜치를 상위로 만들고 원격으로 push를하고 그다음 하위로 gloabal-styles랑, theme를 각각 만들고 여기서부터 각각 feature/1--styles로 pr을 올리고 하위 브랜치 2개가 다 pr이 완료되면 상위브랜치인 feature/1--styles로 병합을 하고 develop으로 pr을 올린다음 여기서는 코드리뷰가 크게 안해도 멘토님이 괜찮다고 하셔서 왜냐하면 하위브랜치 2개를 그냥 상위 브랜치로 병합해서 pr을 올린거기때문에 그래서 큰 오류가 없으면 develop으로 merge하는 과정으로 이해했습니다.

[Donghunn-Lee]

아하 듣고 보니 세부 기능별로도 다 코드리뷰를 할 수 있으면 좋겠네요. 그러면 PR이 빈번해지는 만큼 merge approve를 어떻게 하느냐가 중요할 것 같은데.. 저번에 한 명으로 얘기했듯이 한 명 이상 리뷰를 달면 올린 사람이 merge 하는 식으로 하면 괜찮겠죠?

+px, rem단위에 대해선 제가 프로젝트 계획서중 멘토 피드백 내용이 있어 멘토님게 DM 드렸었는데, 피드백 주실 때 같이 답변해주십사 하고 뭘 사용하면 좋을지 질문 드렸습니다 : ) 답변 주시면 스레드에 알려드릴게요~

[kimdohoon2]

네~알겠습니다!감사합니다~