Feature/apply page interaction#59

[shu07002]

✅ Checklist

• 올바른 브랜치에 PR을 보내도록 설정하였나요?
• PR의 라벨을 올바르게 달았나요?

✨ 현재 작업은 어떤 이슈를 해결한 것인가요?

관련된 이슈 number: #59

• apply 페이지에서 지원 과정에 따라 버튼을 눌러서 넘어가는 과정 구

• 과정 내에서 상단에 존재하는 프로그레스 바 구현

• 모바일 환경일 때, 지원하기 버튼 클릭 시 깨지는 PCOnly Modal 수정

• ScrollLottie에 play={true} 속성을 추가하여 애니메이션이 작동되도록 수정

---

• PCOnly를 apply 페이지 내에서 사용자 화면의 사이즈에 따라 호출하도록 변경
• FAQ에 있었던 answer가 잘리는 문제 해결

기존 코드에서 변경된 사항

• 있음
• 없음

~ 있을 시 내용..

[rmdnps10]

1. 현재는 /recruit에서 지원하기 버튼을 눌렀을 때 PConly 상태변수 여부에 따라서 아래의 UI를 렌더링하는데,
   /apply로 URL로 바로 들어갔을 경우에 제약사항이 없이 들어갈 수가 있어서,

/apply 페이지에 모바일로 들어갔을 경우에 이 UI를 렌더링하는 방향으로 가는 게 좋을 거 같습니다!
그래서 일단 apply now누르면 /apply로 이동하게 하고, apply 안에서 처리를 하는 게 좋을 거 같아용

2. contianer폴더의 PConly.tsx와 components 폴더의 PcOnly.tsx 가 따로 분리되어이 있는 이유가 뭘까요??

[rmdnps10]

PcOnly.tsx나 PConly.tsx로 상태변경함수인 setPcOnly 자체를 남겨주는 것보다는

const togglePCOnly = ()=>{
... 
}

togglePCOnly 처럼 props 으로 넘겨주는 함수의 동작을 구체적으로 지정하는게 practice인 거 같습니다:)

[shu07002]

1. 넵 그렇게 다시 수정하겠습니다!

2. 제가 노트북이 고장나서 윈도우를 아예 다시 처음부터 깔았는데 깃에서 풀 받을때 파일 위치를 잘못 찾아서 한개를 더 만든것 같습니다! 현재 PCOnly(대문자 c)를 사용하기때문에 PcOnly(소문자 c)는 삭제하도록 하도록 하겠습니다!

3. 넵 toggle로 다시 구현해보겠습니다!

[rmdnps10]

이건 PR 내용이랑은 상관없는건데 토글 텍스트가 살짝 짤리는 거 같아용 ㅎㅎ

[shu07002]

헉 height 때문에 짤리고 있었네요 이것도 수정하겠습니다!

[shu07002]

PcOnly.tsx나 PConly.tsx로 상태변경함수인 setPcOnly 자체를 남겨주는 것보다는

const togglePCOnly = ()=>{
... 
}

togglePCOnly 처럼 props 으로 넘겨주는 함수의 동작을 구체적으로 지정하는게 practice인 거 같습니다:)

이거 그냥

이런 식으로 해도 상관 없을까요? 아니면 말씀해주신대로 함수를 통해서 하는게 좋을까요?

[rmdnps10]

PcOnly.tsx나 PConly.tsx로 상태변경함수인 setPcOnly 자체를 남겨주는 것보다는

const togglePCOnly = ()=>{
... 
}

togglePCOnly 처럼 props 으로 넘겨주는 함수의 동작을 구체적으로 지정하는게 practice인 거 같습니다:)

이거 그냥 이런 식으로 해도 상관 없을까요? 아니면 말씀해주신대로 함수를 통해서 하는게 좋을까요?

굳이 상태변수 선언 추가로 하지않고 css로 지정해주시는게 더 좋을 거 같습니다:)

[rmdnps10]

LGTM :) 고생하셨습니다!!!