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

강원대 FE_조중현_1주차 과제 #77

Open
wants to merge 23 commits into
base: faddishcorn
Choose a base branch
from

Conversation

faddishcorn
Copy link

@faddishcorn faddishcorn commented Jun 28, 2024

첫 프로젝트 세팅 시 미처 커밋을 제대로 수행하지 못하였습니다.. 앞으론 상세히 커밋할 수 있도록 하겠습니다..!

Copy link

@lazy-sky lazy-sky left a comment

Choose a reason for hiding this comment

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

마치 어느 정도 경험이 있는 것처럼 전체적으로 능숙하게 잘 작성해주셨네요! 1주차 과제 고생하셨습니다.


### 질문 3. React에서 상태 변화가 생겼을 때 어떻게 변화를 알아챌 수 있나요?

userState같은 훅을 사용하여 상태를 정의하고 이것의 상태 업데이트 함수가 호출되면 리액트는 해당 컴포넌트의 상태가 변경되었음을 인식 및 재 렌더링 합니다. 그에 따라 변경된 상태를 사용하여 UI를 업데이트합니다.

Choose a reason for hiding this comment

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

조금 더 근본적으로 리액트가 상태 변화를 감지하는 과정에 대해서 이해하려면 '리액트는 state를 immutable한다.'는 사실을 이해해야 하는데, 여유가 되실 때 이 개념에 대해 학습하시는 것을 추천드립니다.

argTypes: {
theme: {
control: 'select',
options: ['kakao', 'outline', 'black', 'lightGray', 'darkGray'],

Choose a reason for hiding this comment

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

options에 들어올 수 있는 값들도 타입으로 정의하면 type safety, 자동 완성 등의 측면에서 이점을 얻을 수 있습니다.

import styled from '@emotion/styled';

type Theme = 'kakao' | 'outline' | 'black' | 'lightGray' | 'darkGray';
type Size = 'large' | 'small' | 'responsive';

Choose a reason for hiding this comment

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

아 여기서 타입 선언하신 걸 뒤늦게 봤네요. 그렇다면 export 하여 스토리북 파일에서 재사용하면 좋을 거 같습니다.

Copy link
Author

Choose a reason for hiding this comment

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

넵 재사용해보겠습니다 !

'react/require-default-props': 'off',
'no-unused-vars': 'warn',
'import/no-unresolved': 'off', // 임시로 경고 무시
'import/extensions': 'off', // 임시로 경고 무시

Choose a reason for hiding this comment

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

왜 임시 룰을 적용하게 되셨나요?

Copy link
Author

Choose a reason for hiding this comment

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

확장자 명 관련 오류랑 컴파일할 때 eslint가 import 경로관련 에러를 자꾸 출력해서 관련규칙을 이것저것 꺼보느라 남겨두었던 주석을 그대로 남겨둔 것 같습니다😅

border: 1px solid #ccc;
border-radius: 4px;
${({ size }) => size && sizeStyles[size]}
${({ disabled }) =>

Choose a reason for hiding this comment

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

조건부 스타일을 잘 적용하셨네요!

@lazy-sky
Copy link

lazy-sky commented Jul 2, 2024

base 브랜치를 변경해주세요!

@faddishcorn faddishcorn changed the base branch from main to faddishcorn July 2, 2024 09:30
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