-
Notifications
You must be signed in to change notification settings - Fork 11
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
9회차과제 - 김주희 #8
base: main
Are you sure you want to change the base?
9회차과제 - 김주희 #8
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.
코드도 전반적으로 깔끔하게 쓰신거 같고 다양한 폰트, 로고 (svg)를 쓰신 거 같습니다! 확실히 눈에 들어오고 이쁘네요:) 수고하셨습니당
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.
svg 파일을 사용할때 publice에 정리하는 것이 컴포넌트를 더 깔끔하게 보일 것 같습니다. import { ReactComponent as MyIcon } from './icons/my-icon.svg'; 이런식으로 하면 될거같습니다!
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.
오! 좋은 방법인 것 같습니다~ 참고하겠습니다!
리뷰 감사합니다:)
<Route path="/about" element={<About/>}/> | ||
<Route path="/techService" element={<TechService/>}/> | ||
<Route path="/notice" element={<Notice/>}/> | ||
<Route path="*" element={<NotFound/>}/> |
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.
<Route path="*" element={}/> 사용하여서 error 페이지 만든 것이 인상깊네요!
내가 개발한 기능
유의 깊게 개발한 부분
개발할 때 들었던 의문사항