-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FIX] QA 대응 - 로그인 리다이렉트 및 데탑 뷰 대응 #402
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
어우 css 고생많으셨습니다 👍
브라우저 크기대응되는거 속이 다 시원하네욤 ㅋㅋㅋ
src/styles/GlobalStyle.tsx
Outdated
html { | ||
font-size: 62.5%; | ||
font-size: 45%; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
얘가 45%가 되면 기본 폰트 크기가 10px이 되는 62.5와 다르게 피그마 보고 rem 단위로 css 작성하기가 번거로울 것 같은데, 좋은 대비책이 있을까요? 기존에 작성한 값들 기준으로 가서 별 상관없나요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
기존에 모든 폰트 크기를 rem 단위로 작성했기 때문에 font-size의 % 값을 조정하는 것이 비율적으로 큰 영향을 주지는 않겠지만, 현재 디자인이 1920px * 1080px을 기준으로 제작되었기 때문에 font-size의 % 값을 줄이지 않으려면 피그마에서 설정된 디자인 크기 자체를 수정하는 것이 필요할 것으로 보입니다..!
디자인 크기의 즉각적인 수정이 어렵기 때문에, 우선 임시 방편으로 font-size: 45%로 설정해두었습니다.
src/pages/Login.tsx
Outdated
const isAuthenticated = localStorage.getItem('accessToken'); | ||
const naviate = useNavigate(); | ||
|
||
useEffect(() => { | ||
if (isAuthenticated) { | ||
naviate('/today'); | ||
} | ||
}, [isAuthenticated, naviate]); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
굿굿 따로 훅으로 빼도 좋을 것 같네요!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
해당 기능 useAuthRedirect 훅으로 분리했습니다! 훨씬 코드가 깔끔해지고 좋으네용 좋은 의견 감사합니다!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
작업 내용 🧑💻
알게된 점 🚀
리뷰 요구사항 💬
관련 이슈
close #401
스크린샷 (선택)
https://github.com/user-attachments/assets/e871b0a0-063a-4db8-9b45-bc9644c98d9c