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

5주차 Assignment - 안준영 #5

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

Conversation

Junyeong-An
Copy link

@Junyeong-An Junyeong-An commented Nov 2, 2023

image

클론코딩 과제 제출

Copy link

@GraceKim527 GraceKim527 left a comment

Choose a reason for hiding this comment

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

과제 통과
수고하셨습니다!!

text-decoration: none;
margin: 0 10px;
}
.etc {

Choose a reason for hiding this comment

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

justify-content 및 align-items의 경우는 display: flex; 가 있어야 작동 가능합니다!! 단독으로는 사용할 수 없다는 점 참고해주세요!

border: 0;
}

select {

Choose a reason for hiding this comment

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

피드백

image

이렇게 특정 아이템이 선택했을 때 border자체는 없지만, 파란색 줄 처럼 outline이 남아 있는 경우가 있어요! 이부분은 outline: none; 혹은 0으로 처리해주시면 좋을 것 같습니당

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.

2 participants