Skip to content
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

임예솔 과제 제출합니다 #1

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

sonamulovebunny
Copy link

@sonamulovebunny sonamulovebunny commented Oct 11, 2024

과제 제출일을 헷갈려서 급하게 제출했습니다....
코어님께 icon 관련 질문 남겨놓은 상태라 답변 받으면 과제3 JS 추가해서 재제출하겠습니다.....

Copy link

@leemanjae02 leemanjae02 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • 본인이름 폴더를 만들고 그 안에 과제를 작성해주세요. 이미지 폴더도 별도로 만드셔야합니다.
    화면 중앙정렬을 계속 margin값을 조정해서 중앙정렬처럼 보이게 하시는데 따로 이유가 있나요?
    중앙 정렬을 만들어야 하는데 margin값을 조정해서 임의로 그렇게 보이게 만든다면 나중에 화면에 배치할 요소가 많을 때 레이아웃 수정이 힘들어집니다.
    그리고 JS코드에 대한 동작을 주석으로 적어주시기 바랍니다.

  • 과제 1번은 해결 방안을 알려드렸으니 아이콘이 보이게 수정하시면 됩니다.
    또한 공백을 입력하고 넘어가도 로그인 성공창으로 넘어가네요. 공백 입력 방어를 추가하셔야 합니다.

  • 과제 2번 로그인 버튼의 활성화 상태일 때 글자색을 흰색으로 설정해주세요.

  • 과제 3번은 아직 구현이 다 되진 않았네요 주어진 조건에 맞춰 수정 후 다시 제출해주세요.

@sonamulovebunny
Copy link
Author

sonamulovebunny commented Oct 15, 2024

과제 1,2 수정했습니다.
3은 너무 어려워서 조금 더 공부하고 재제출 하겠습니다.

@tndls0302
Copy link
Collaborator

제출 완료.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants