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

Week02/chaeyeon #10

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

Week02/chaeyeon #10

wants to merge 15 commits into from

Conversation

qkrcodus
Copy link
Contributor

@qkrcodus qkrcodus commented May 20, 2024

✨어떤 과제를 수행했나요?✨

image

image

image

✨어려웠던점, 트러블슈팅✨

<캘린더>
기존 html 코드에 날짜를 임의로 넣어두었는데 ,해당 월 1일의 요일, 해당 월의 요일 수를 담아두는 변수를 만들고, renderCalendar 함수에서 해당 월의 요일 수 만큼 li태그를 넣어주는 방식으로 바꿨다.
현재 날짜를 강조하기 위해 li태그를 넣어주는 코드에서 조건문을 이용하여 특정 class 를 갖는 li태그를 넣어주었고 css에서 속성을 입혀주었다.

<당근마켓>
js를 이용하지 않고 html css 로만 작아진 화면에서 버튼을 눌렀을때 nav바의 토글기능이 되게 해주기 위해서 input type="checkbox" 와 그 뒤에 label 태그를 넣어주어 checkbox뿐만 아니라 label을 클릭해도 모달 창이 뜨게 클릭 영역을 넓혔다.
토글 기능은 아직 구현 중이다!

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