학생 페이지의 취업현황 탭 구현, MOU가 체결되지 않은 기업 등록, 기업에 학생 수동 취업 처리 기능구현 #169
Conversation
1408bg
requested review from
jikwan0327,
KANGYONGSU23 and
JJIIIINN
as code owners
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
1408bg
requested review from
dnjsdmswl and
eejx0
and removed request for
KANGYONGSU23,
JJIIIINN and
jikwan0327
seungw0o
requested review from
phyuna0525,
JJIIIINN,
jikwan0327 and
KANGYONGSU23
|
@1408bg 커밋 메세지가 이상하게 표시되는데 확인 한번 해주시면 좋을 거 같습니다 |
|
git log 찍으니까 한국어로 나오는데 깃허브에서 보니까 인코딩이 깨져있네요 |
|
@1408bg |
| --subcolor-red-10: #4c1914; | ||
| --subcolor-red-20: #e74c3c; | ||
| --subcolor-green-10: #0f4325; | ||
| --subcolor-green-20: #2ecc71; | ||
| --subcolor-blue-10: #0c2942; | ||
| --subcolor-blue-20: #237bc9; | ||
| --subcolor-yellow-10: #504105; | ||
| --subcolor-yellow-20: #f1c40f; |
There was a problem hiding this comment.
red, yellow, green은 없어지는 건가요? 토스트에서 사용하고 있던 걸로 알고 있어서요
There was a problem hiding this comment.
그리고 색상 변경할 때 company에도 영향이 가기 떄문에 같이 확인 부탁드려요
There was a problem hiding this comment.
red, yellow, green은 없어지는 건가요? 토스트에서 사용하고 있던 걸로 알고 있어서요
실수로 다른 subcolor를 지웠었습니다.
수정했습니다.
그리고 색상 변경할 때 company에도 영향이 가기 떄문에 같이 확인 부탁드려요
색상 변경은 디자인에서 수행되었고, 저는 해당 변경사항을 반영했습니다.
디자인 상에서 색상과 함께 변경이 명시된 컴포넌트는 Footer밖에 없습니다.
company에서 디자인과 색상의 일치 여부를 모든 페이지에서 확인하면 될까요?
| `; | ||
|
|
||
| const StyleInput = styled(Input)` | ||
| width: calc(54vw - 249px); |
There was a problem hiding this comment.
width를 이렇게 지정하면 화면 너비에 따라 레이아웃이 깨지는 현상이 자주 발생할 거 같아요. 해당 input의 width를 지정할 때는 viewport가 아닌 %나 상위 레이아웃의 너비가 고정이라면 px로 지정하는 것을 권장드려요
There was a problem hiding this comment.
@jobis/ui의 Input 컴포넌트에서 내부적으로 div를 하나 더 쓰고있기에, %로 지정하면 202px에서 늘어나지 않습니다.
디자인상 717px인데 px로 수정할까요?
| if (studentGcns.filter(e => e !== -1).length === 0) { | ||
| navigate("/"); | ||
| } | ||
| mutate(); |
There was a problem hiding this comment.
혹시 의도하신 동작인지는 모르겠지만 해당 조건문에 걸리는 여부와 상관없이 navigate가 되더라도 mutate가 실행될 수도 있을 거 같아요. 만약 조건에 걸렸을 때 mutate를 실행하고 싶지 않다면 조건문에 함수를 종료하는 return을 추가하는 것이 더 안전할 거 같네요
| const FileInput = styled.div` | ||
| display: flex; | ||
| justify-content: space-between; | ||
| width: calc(54vw - 200px); |
최대한 gap을 사용하도록 변경하겠습니다.
|
What is this PR?🔍
#168이슈 '학생 페이지의 취업현황 탭 구현, MOU가 체결되지 않은 기업 등록, 기업에 학생 수동 취업 처리 기능구현' 을 해결합니다.작업내용📑
스크린샷📸
Issues