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.
Related Issue : Closes #372
🧑🎤 Summary
반응형 마이페이지를 구현했습니다
🧑🎤 Screenshot
2024-08-10.9.41.13.mov
RPReplay_Final1723294522.MP4
2024-08-10.9.56.37.mov
🧑🎤 Comment
🚀 마이페이지 반응형 작업
다 styleVary만 쳐주면 되는거라 무리없이 작업했습니다
(반응형 요소들 정리한 노션)
🚀 ReviewPage 접속 제한
전에 이야기했던 대로 ReviewPage는 우선 반응형 작업 미루고, 따라서
지원서 확인
클릭 시 사용자 상태를 체크해서 접속 제한 처리를 해줬는데요,두가지 케이스가 있어요
이때 모바일로 접속하면 무조건 'PC로 접속하라'는 alert 띄우고 이동을 막으면 되는데,
PC에서 작은 창으로 접속할 경우에도 PC로 접속하라는 alert을 띄우는게 부적절하다고 생각해서 '전체화면 이용을 권장드려요' 라고 alert 문구를 바꿔줬는데요,
이와 동시에 이렇게 권장드린다는 말을 하고나서 이동까지 아예 막아버리는 것보다, 그냥 사용자가 알아서 창 크기 키우겠거니 하고 페이지 이동은 정상적으로 진행시키는게 낫다는 생각이 들었어요.
그래서 정리하자면
이렇게 일단 구현해주었습니다.
제가 임의로 정한거라, 이견 있으시면 반영할게요!!
🚀 Button padding left, right 0 처리
깨지고 있는 letterSpacing까지 제대로 다시 넣어줬는데도 모바일 화면에서 줄바꿈이 발생하길래 왜지 했는데
피그마 상에서 padding 영역이 이렇더라고요?
영역을 제대로 안보고 padding을 15, 25로 주니까 깨졌던 거였고,
이메일 버튼 깨짐 이슈 때처럼 padding left, right 0으로 줘서 해결했어요