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

seonmiki 개포동배달음식, TODO, web(2), react 로그인 #14

Open
wants to merge 8 commits into
base: seonmiki_main
Choose a base branch
from

Conversation

seonmiki
Copy link

@seonmiki seonmiki commented Aug 13, 2024

까비 온보딩

  • JavaScript -> LV 1. 개포동 배달음식은 비싸다
  • HTML + CSS + JavaScript -> LV 1. Todo List 마스터하기
  • HTML + CSS + JavaScript -> LV 2. Cabi 웹 페이지 만들기(2)
  • React -> LV 1. 그냥 모르는데요 …?

제출합니다 :)

@seonmiki seonmiki changed the title seonmiki 개포동배달음식, TODO seonmiki 개포동배달음식, TODO, web(2) Aug 26, 2024
@seonmiki seonmiki changed the title seonmiki 개포동배달음식, TODO, web(2) seonmiki 개포동배달음식, TODO, web(2), react 로그인 Sep 8, 2024
@seonmiki seonmiki self-assigned this Sep 8, 2024
Copy link

@Minkyu01 Minkyu01 left a comment

Choose a reason for hiding this comment

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

REACT_LV_1 은 혼합 Styled-components로 보기 좋게 만들어져 있네요!! 고생하셨습니다

width: 500px;
height: 60px;
border-radius: 20px;
padding-left: 30px;

Choose a reason for hiding this comment

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

input에도 글자수 제한이 있으면 좋겠네요~!
padding : 0 30px;
스크린샷 2024-09-13 오후 12 26 19

width: 50%;
height: 60vh;
border-radius: 20px;
overflow-y: scroll;

Choose a reason for hiding this comment

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

각 article에 스크롤 바가 생기는 것은 의도한 부분인가요?? 만약 그렇지 않다면 auto로 설정하면 내부 요소가 넘치진 않을 때 스크롤바를 없애고, 넘칠때만 스크롤바를 생성되게 할 수 있습니다!!

}

p {
font-family: "Do Hyeon", sans-serif;

Choose a reason for hiding this comment

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

font-family도 body에서 font-family: "Inter", sans-serif, "Do Hyeon";을 먼전 선언한 다음해당 부분에서
font-family: "Do Hyeon"을 선언하면 font-family가 중복되는 것을 방지할 수 있습니다

cursor: not-allowed;
}


Choose a reason for hiding this comment

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

반응형 고려하실때 모바일 반응형은 적용이 안되있는거 같습니다... 시간 되실 때 해보시면 도움이 많이 되실 거예요!

Copy link

@Minkyu01 Minkyu01 left a comment

Choose a reason for hiding this comment

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

HTML+CSS_LV_1_Cabi_웹_페이지_만들기 파일이 잘못 올라간거 같습니다. nav와 전반적인 크기가 깨집니다. 확인 부탁드려요~!

body {
line-height: 1;
}
/* ol,

Choose a reason for hiding this comment

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

주석을 해서 li에 marker가 나오게 되었네요

Copy link
Author

Choose a reason for hiding this comment

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

헉 죄송합니다! 이전 파일 확인하면서 건드린 게 잘못 올라간 것 같습니다ㅠㅠ 수정해서 변경해두었습니다 확인해주셔서 감사합니다!!

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