Conversation
sheepyis
approved these changes
Jan 10, 2026
Member
sheepyis
left a comment
sheepyis
left a comment
There was a problem hiding this comment.
- 전체적으로 any 타입은 지양하는 방향으로 다 수정 부탁드립니다.
- 나중에 기수 대표 색상이 바뀌었을 때 한번에 관리하기 쉽도록, 색상이나 그라데이션 등은 styles>theme.css에 정의한 것을 사용하는 방식으로 부탁드립니다.
- 대부분 웹 이미지와 앱 이미지를 별도로 다운 받아 사용하신 것 같은데, svg 이용하는 방향이면 더 좋을 것 같습니다!
- 반응형과 관련하여 430px 기준으로 나눠서 구현하신 것 같습니다. 다만 이렇게 진행하면, 430px 기준으로 자연스럽지 않고 뚝 끊겨 보일 수 있어 다른 분들 코드처럼
text-[clamp(1.75rem,4vw,3.6rem)]의 방식으로 부탁드립니다! - 파일명이 대소문자가 섞여 일정하지 않은 규칙으로 작성되어 보이는데, 컴포넌트의 경우 무조건 파스칼케이스 형식으로 하고, 이미지의 경우 카멜 케이스로 부탁드립니다!
- 전체적으로 시맨틱 태그 이용해서 코드 짜신 점이 인상 깊었습니다. 코드 리뷰 내용을 1차적으로 반영해서 수정 부탁드리며, 미리 merge 하게 되면 브랜치가 삭제되어 다시 생성 및 pr 하셔야 해서.. 수정 후 Merge 부탁드립니당!
Member
There was a problem hiding this comment.
웹과 앱의 사진 두 개를 넣기보다 svg 파일 하나로 재사용 하는 방향으로 수정해주세요!
Member
There was a problem hiding this comment.
모든 아이콘들도 웹과 앱 두 개의 사진 파일 사용하지 말고, svg 하나로 크기 조정해서 사용해주세요!
| setRecruitData(RecruitData); | ||
| }, []); | ||
|
|
||
| const handleButtonClick = () => { |
Member
There was a problem hiding this comment.
useNavigation 커스텀 훅 만들어놓은 걸 사용하면 더 좋을 것 같습니다!
src/pages/recruit/Recruit.tsx
Outdated
| }, []); | ||
|
|
||
| const handleButtonClick = () => { | ||
| window.open( |
Member
There was a problem hiding this comment.
이 부분도 마찬가지로, useNavigation 커스텀 훅 사용해주세요!
Member
There was a problem hiding this comment.
- 링크들을 별도의 더미데이터로 분리하면 나중에 링크 수정할 때 파일 찾기 더 편할 것 같습니다.
src/utils/Recruit/loadImages.ts
Outdated
| type GlobImport = Record<string, { default: string }>; | ||
|
|
||
| const imageImport = import.meta.glob( | ||
| "../../assets/images/Recruit/Competency/*.png", |
src/utils/Recruit/loadImages.ts
Outdated
| export const images = Object.keys(imageImport).reduce<Record<string, string>>( | ||
| (acc, key) => { | ||
| const path = key | ||
| .replace("../../assets/images/Recruit/Competency/", "") |
src/utils/Recruit/recruitData.ts
Outdated
| { | ||
| activeDate: [{ id: 1, title: "2026.03 ~ 2026.08 [약 6개월]" }], | ||
| part: [ | ||
| { id: 1, title: "PM", explain: "프로젝트 일정 관리 및 기획" }, |
src/utils/Recruit/recruitData.ts
Outdated
| { | ||
| id: 1, | ||
| step: "서류 모집", | ||
| date: "2024.09.02(월) ~ 09.08(일)", |
Member
There was a problem hiding this comment.
피그마에 맞춰서 0000.00.00(월) ~ 0000.00.00(일) 형식 맞춰주시면 감사하겠습니다!
src/utils/Recruit/recruitData.ts
Outdated
| { id: 6, title: "Android", explain: "앱 개발(Android)" }, | ||
| { id: 7, title: "iOS", explain: "앱 개발(iOS)" }, | ||
| ], | ||
| people: [ |
|
🚀 Deployed on https://6966185fde26388bbe5995ee--hsu-umc.netlify.app |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🚀 관련 이슈
🔑 작업 내용
Recruit 페이지 UI 구현
📷 스크린샷
🌐 공유 사항 to 리뷰어
<!— 리뷰어가 중점적으로 봐줬으면 좋겠는 부분이 있다면 적어주세요 —>
<!— 논의해야 할 부분이 있다면 적어주세요 —>
리팩토링 하겠습니다
🚨 이슈 사항
<!— 이슈가 발생하는 부분이 있다면 적어주세요 —>