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

[Yunjanghyeon] 프론트엔드 1주차 미션 제출합니다. #8

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Yunjanghyeon
Copy link

No description provided.

Copy link
Collaborator

@hhhkdev hhhkdev left a comment

Choose a reason for hiding this comment

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

내용 정리를 굉장히 많이하셧네요..!

정리한 내용들 중에 좋은 내용이 많았어서 얻어가는게 많앗으면 좋겟습니당 2주차 미션에서도 내용 정리하는 것들이 있긴한데 이보다 코드가 더 중요하니 너무 정리에 치중하실 필요없을거 같아용

Comment on lines +1 to +6
div
{border-radius:0px 40px 40px 40px; width : 500px; height: 100%; text-align: center;font-size: 20px; background-color: gray; font-family: '맑은 고딕', 'Arial Narrow', Arial, sans-serif;border: 3px solid black;}
a:hover
{text-decoration: none; background-color: white;}
h3
{width:75px; border: 3px solid black; background-color:gray;text-align:left;}
Copy link
Collaborator

Choose a reason for hiding this comment

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

hover 속성까지 이용한게 진짜 인상깊긴한데

코드가 정렬이 안되어있어서 보기가 쉽지 않네요... 이럴 경우에는 코드를 짜기에도 난항을 겪을 수 있어서 prettier 등 코드 포맷팅해주는 vs code extension 찾아서 이용해보시면 좋을거 같아여

Comment on lines +14 to +25
<div>
역할: 프론트엔드<br>
거주지: 서울시 영등포구<br>
MBTI: ESFP<br>
깃허브: <a href ="https://github.com/Yunjanghyeon">깃허브 링크</a><br>
블로그: <a href="https://blog.naver.com/yjh_zz">블로그 링크</a><br>
생일: 2004년 10월 6일<br>
좋아하는 음식: 대부분<br>
취미: 게임, 야구/축구 관람,음악 듣기<br>
디스코드 닉네임: 윤장현<br>
시즌: 2024-2<br>
</div>
Copy link
Collaborator

Choose a reason for hiding this comment

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

자기소개를 조금더 풍성하게 만들엇으면 좋을거 같아여 HTML태그도 조금더 여러개 이용해보는 경험해보면 좋을거 같습니당

Comment on lines +21 to +22
var change1 = +cal[0]
var change2 = +cal[2]
Copy link
Collaborator

Choose a reason for hiding this comment

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

cal에 들어있던 값들을 미리 변수에 담아놓아야 관리하기 쉬울거 같습니다..!

그리고 +를 붙인건 문자를 숫자로 바꾸기 위함으로 보이는데, parseInt도 알아보시면 좋을거 같아영 var 선언도 if문 내부가 아니라 밖에서 하면 아래 cal[0], cal[2]로 접근한 것들을 변수로 접근할 수 있지않을까 싶습니다

@@ -0,0 +1,8 @@
function Button(word){
Copy link
Collaborator

Choose a reason for hiding this comment

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

약간은 prop이라는 개념에 대해 이해해보면 좋을거 같아요 물론 3주차 미션에서 배우겠지만..

리액트에서 컴포넌트 외부에서 내부로 데이터를 전달한 것들을 props라 해요. 지금 word가 바로 그 객체인데, 이 안에 컴포넌트를 이용하여 전달한 값들이 담기게 됩니당 지금은 text라는 속성이 담겼네요? 이걸 아래처럼 출력하는 형식이에여

물론 이름을 word로 한다고 달라지는건 없다만 웬만해선 props의 개념에 대해 알고 쓰면 좋을거 같스빈당

<div className="App">

<h1>인사해요</h1>
<Button text={'ㅎㅇ'}/>
Copy link
Collaborator

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