-
Notifications
You must be signed in to change notification settings - Fork 79
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
[김유정] Sprint7 #623
The head ref may contain hidden characters: "React-\uAE40\uC720\uC815-sprint7"
[김유정] Sprint7 #623
Conversation
…ithub-actions [Fix] delete merged branch github action
…-sprint1 [김유정] Sprint1
… into part1-김유정-sprint1
…정-sprint2 [김유정] Sprint2
…정-sprint3 [김유정] Sprint3
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.
@@ -0,0 +1,23 @@ | |||
module.exports = { |
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.
eslint를 설정하셨군요 👍 👍
|
||
function Button({ children }) { | ||
return ( | ||
<> |
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.
[수정해주세요]
여기서 Fragment는 지우셔도 됩니다
import Profile from "../assets/images/profile.png"; | ||
import "../styles/Header.css"; | ||
import { Link, NavLink } from 'react-router-dom'; | ||
import Logo from '@/assets/images/logo.png'; |
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.
absolute path를 사용하셨군요 👍
}`; | ||
|
||
return ( | ||
<> |
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.
[수정해주세요]
여기서 Fragment는 없어도 됩니다!
name="item_name" | ||
placeholder="상품명을 입력해주세요" | ||
/> | ||
<div className={styles.label}>상품 이미지</div> |
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.
[참고해주세요]
여기서는 div대신 label이 더 잘어울립니다!
그래야 '상품 이미지'를 클릭했을때 input에 focus가 생깁니다.
요구사항
기본
심화
주요 변경사항
스크린샷
멘토에게