-
Notifications
You must be signed in to change notification settings - Fork 44
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
[이진욱] Week2 #3
The head ref may contain hidden characters: "part1-\uC774\uC9C4\uC6B1-week2"
[이진욱] Week2 #3
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. svg, png 다양한 이미지를 사용하신 것 같아요. 어떤 기준에서 나눠서 사용하셨는지 궁금해요!
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 이미지 파일 사용 기준은, 픽셀의 왜곡이나 손상이 있어선 안되는 회사 로고 같은 중요 부분들, 사용할 때 직접적으로 클릭하는 등의 상호작용이 필요한 버튼 등의 아이콘을 중점적으로 svg를 사용했고 나머지 조금 덜 중요하다 싶은 부분은 png 로 썻습니다! 더미 파일들은 변명의 여지가 없네요! 다음 주차때 꼼꼼히 확인해서 바로 반영해보겠습니다! ㅠ |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,19 @@ | ||
export const $eyes = document.querySelector('.password_eyes'); | ||
export const $password = document.querySelector('#passwordToggle'); | ||
export const $passwordRe = document.querySelector('#passwordToggleRepeat'); | ||
|
||
const handlePasswordSight = () => { | ||
if ($password.type === 'password') { | ||
$password.type = 'text'; | ||
$eyes.src = '/asset/eye-off.svg'; | ||
$eyes.classList.add('eyes_offset'); | ||
$passwordRe.type = 'text'; | ||
} else if ($password.type === 'text') { | ||
$password.type = 'password'; | ||
$eyes.src = '/asset/eye-on.svg'; | ||
$eyes.classList.remove('eyes_offset'); | ||
$passwordRe.type = 'password'; | ||
} | ||
}; | ||
|
||
export default handlePasswordSight; |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,20 @@ | ||
import { $password, $passwordRe } from './eyesOnOff.js'; // 패스워드 인풋창이랑 패스워드 재입력 인풋창 | ||
const $wrongMessage = document.getElementById('wrong_message'); // 비밀번호가 틀렸습니다 메세지 div | ||
// const $signUpButton = document.querySelector('.signup_button'); | ||
|
||
const valueWrong = (e) => { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe 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 commentThe 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 commentThe 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 commentThe reason will be displayed to describe this comment to others. Learn more. 넵 나중에 파일이 많아질 것을 대비하여 '목적에 맞게 분리한다'정도로 우선 가져가셔도 좋을 것 같아요. 추후에 React 등 프론트엔드 라이브러리 혹은 프레임워크를 사용하시다보면 자연스럽게 학습하게되시는 부분도 있으니 지금 너무 고민 안하셔도 될 것 같긴합니다 :) |
||
e.preventDefault(); | ||
if ($passwordRe.value) { | ||
if ($passwordRe.value !== $password.value) { | ||
$wrongMessage.classList.remove('hidden'); | ||
$passwordRe.className = 'wrong_password'; | ||
} else { | ||
$wrongMessage.className = 'hidden'; | ||
$passwordRe.classList.remove('wrong_password'); | ||
} | ||
} | ||
}; | ||
|
||
// const ifWrongValue = () => {}; | ||
|
||
export default valueWrong; |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,11 @@ | ||
<!DOCTYPE html> | ||
<html lang="en"> | ||
<head> | ||
<meta charset="UTF-8" /> | ||
<meta name="viewport" content="width=device-width, initial-scale=1.0" /> | ||
<title>Document</title> | ||
</head> | ||
<body> | ||
faq | ||
</body> | ||
</html> |
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.
DS_Store는 Desktop Services Store 의 약자로, 애플에서 정의한 파일 포맷입니다.
애플의 맥OS X 시스템이 폴더에 접근할 때 생기며, 해당 폴더에 대한 메타데이터를 저장하는 파일입니다. 즉, 이 부분은 git에 따로 올라갈 필요가 없는 파일이에요! 추후 .gitignore에 대해서도 보실 수 있을텐데, https://www.toptal.com/developers/gitignore
이를 통해 git에 올라가지 않는 파일들을 한 번에 관리할 수 있으니 참고해주세요 :)
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.
깃헙 사용법에 익숙치 않아 엄한곳에서 시간을 쓰다보니 정작 이러한 부분을 놓쳤네요. 다음부턴 주의 하겠습니다!