-
Notifications
You must be signed in to change notification settings - Fork 3
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
[ Login ] 로그인 뷰 구현 #44
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.
시멘틱 이주함 선생
@@ -1,5 +1,9 @@ | |||
import { St } from "./style"; | |||
|
|||
export default function CTABtn() { | |||
return <St.Button to="/category">대화주제 추천 바로가기</St.Button>; | |||
return ( | |||
<St.Button role="button" to="/category"> |
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.
왜 type이 아니라 role이죠 ??
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.
button 태그가 아니라 Link 태그의 요소이고, 스크린리더에서 읽히는 role 을 명시하기 위해 추가하였습니다!
src/components/Main/MenuBar/style.ts
Outdated
/* width: 36rem; | ||
height: 100vh; */ | ||
|
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.
주석 지워주시와유
// return ( | ||
// <St.HeaderWrapper> | ||
// <Link to="/"> | ||
// <IcLogo aria-label="피클" /> | ||
// </Link> | ||
// <St.HamburgerContainer isClicked={isOpened}> | ||
// <IcHamburger aria-label="메뉴" onClick={handleModal} /> | ||
// </St.HamburgerContainer> | ||
// </St.HeaderWrapper> | ||
// ); | ||
|
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.
😎
// ); | ||
|
||
useEffect(() => { | ||
console.log(isOpened); |
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.
콘솔로그도 지워줘도 되지 않나용 ?~?
📌 내용
errorMessage
state 로 통해 이메일, 비밀번호 다를 때에 메시지를 나타냅니다📌 내가 알게 된 부분
--> UX 적으로 디자인팀에서 합의한 내용
📸 스크린샷