[Feat] 학생용 페이지 모바일 대응 #171
Conversation
SeieunYoo
requested review from
eugene028,
hamo-o and
soulchicken
as code owners
apps/client/middleware.ts
Outdated
|
|
||
| if (!accessToken) { | ||
| return NextResponse.redirect(new URL(routePath.auth, req.url)); | ||
| } | ||
|
|
||
| if (isMobile && (url.pathname === "/my-study" || url.pathname === "/auth")) { |
There was a problem hiding this comment.
P3;
요렇게 코드가 작성되면 왜 my-study랑 auth만 비교하지..?! 라고 맥락 리소스가 쌓일 것 같아서,
모바일을 허용하는 링크라는 변수로 따로 빼서 include와 같은 메소드로 비교하는 게 좋을 거 같아요!
There was a problem hiding this comment.
isMobileAllowedUrl 이라는 유틸로 분리했습니다! 03098e8
apps/client/app/(afterLogin)/(mobile)/mobile/announcement/page.tsx
Outdated
Show resolved
Hide resolved
apps/client/app/(afterLogin)/(mobile)/mobile/attendance-check-info/page.tsx
Outdated
Show resolved
Hide resolved
| placeholder="Ex. 0000" | ||
| value={attendanceNumber} | ||
| textareaProps={{ | ||
| onKeyDown: handleKeyDown, |
There was a problem hiding this comment.
P3;
저의 개발환경에서는 출석체크 입력 테스트를 할 수 있는 환경이 아니여서 확인은 못해보았지만,
혹시 onKeyDown 이 모바일에서 제대로 동작하는지 체크해주실 수 있을까용?
simulator를 활용하면 확인 가능할 듯 합니다!
There was a problem hiding this comment.
텍스트 필드 키보드를 number 제한하도록 수정하면서 키보드 이벤트에 대한 처리가 필요하지 않을 거 같아 지웠습니다!! 201e1c0
There was a problem hiding this comment.
P3;
커리큘럼 페이지에서 커리큘럼 상세를 클릭하면 PC 화면으로 이동하는데, 모바일에서는 뷰가 깨져 보여용.
그래서 클릭하면 "PC 환경에서 상세 내용을 확인해주세요" 와 같은 alert을 띄운뒤, 모바일에서는 커리큘럼 제목하고 개요만 볼 수 있게끔 아래 화면 유지하는게 어떨까 싶습니다!
There was a problem hiding this comment.
피그마 댓글 확인했을 때 과제 페이지로 이동할 필요가 없다고 해서 링크로 연결되는 부분 지웠습니다!
|
@eugene028 리뷰 반영했습니다!! |
🎉 변경 사항
🚩 관련 이슈
🙏 여기는 꼭 봐주세요!
/mobile/~로 라우팅 해서 대응을 진행합니다.