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

[33기 박슬기] Mission 1 ~ 4 완료 #288

Closed
wants to merge 4 commits into from
Closed

Conversation

DevSeulgi
Copy link

@DevSeulgi DevSeulgi commented May 4, 2022

:: 최근 작업 주제

  • 기능 추가
  • 리뷰 반영
  • 리팩토링
  • 버그 수정
  • 컨벤션 수정

:: 구현 목표

  • westagram 디자인 및 기능 구현

:: 구현 사항 설명

  1. login 버튼 활성화/비활성화 처리
  • ID / PW 부분에 입력 값이 없으면 비활성화 및 curson: not-allowed;
  • ID / PW 창 모두 입력되어 있으면 login 버튼 활성화 처리되며, hover 시 curson: pointer; 로 변경
  1. 댓글 추가 기능 구현

:: 성장 포인트

  • 직접 코드를 짜보니 flexbox에 대해 더 잘 알 수 있게 되었습니다.
  • 레이아웃을 먼저 잡아 놓은 후 css 작업을 진행하는 것이 좋다는 걸 깨달았습니다.
  • z-index를 통한 우선순위를 설정할 수 있다는 것을 배웠습니다.
  • JS 파일은 꼭 body태그 밑 부분에 넣어야 오류가 발생하지 않는다는 걸 배웠습니다.
  • 바닐라JS로 댓글 기능 구현 가능하나 jQuery로도 짧은 코드로 가능하다는 걸 배웠습니다.

:: 기타 질문 및 특이 사항

  • login 버튼 비활성화 처리를 button disabled를 통해 진행하려 했으나 적용되지 않아 css에 button:disabled 속성을 추가하였습니다. 이렇게 진행하여도 문제 없는지 궁금합니다!
  • Live Server 통해 확인하고 있는데, 첫 진입 시 로그인 버튼 기능 비활성화 -> 활성화 전환되는 것은 확인 가능하나, 입력창 내 값 삭제 시 다시 비활성화가 되지 않습니다. 어떻게 해결 가능할까요?

@DevSeulgi DevSeulgi changed the title [33기 박슬기] Add : 기존 코드 옮김 [33기 박슬기] Add : login JS 기능 구현 May 5, 2022
@DevSeulgi DevSeulgi changed the title [33기 박슬기] Add : main JS 댓글 추가 기능 구현 [33기 박슬기] Mission 1 ~ 4 완료 May 6, 2022
@yeonuk-hwang yeonuk-hwang deleted the feature/seulgi branch May 13, 2022 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants