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

04-architecture -> fast-refresh.mdx #433

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

Conversation

xoxwgys56
Copy link

기여자용

문서 개선

Progress

@github-actions github-actions bot added the documentation 문서 작업 label Oct 5, 2023
@g2hhh2ee g2hhh2ee linked an issue Oct 10, 2023 that may be closed by this pull request
### 문법 오류

개발 중에 문법 오류가 생겼다면, 수정하고 파일을 다시 저장하세요.
오류는 자동으로 사라지기 때문에, 앱을 다시 리로드하지 않아도 됩니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
오류는 자동으로 사라지기 때문에, 앱을 다시 리로드하지 않아도 됩니다.
오류는 자동으로 사라지기 때문에, 앱을 다시 로드하지 않아도 됩니다.

표현이 중복되어 수정했습니다 😊

Comment on lines 46 to 47
컴포넌트 내에서 실수로 런타임 오류가 발생했다면, 컨텍스트 오버레이(contextual overlay)가
표시됩니다. 오류를 수정하면 오버레이는 사라지고, 앱을 리로드하지 않아도 됩니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
컴포넌트 내에서 실수로 런타임 오류가 발생했다면, 컨텍스트 오버레이(contextual overlay)가
표시됩니다. 오류를 수정하면 오버레이는 사라지고, 앱을 리로드하지 않아도 됩니다.
컴포넌트 내에서 실수로 런타임 오류가 발생했다면, 상황에 맞는 오버레이가
표시됩니다. 오류를 수정하면 앱을 다시 로드하지 않아도 자동으로 오버레이가 자동으로 해제됩니다.

'앱을 다시 로드하지 않아도'를 강조하면 사용자들이 내용을 더 잘 파악할 수 있을 것 같아요. 이것이 문서의 명확성을 높이는데 도움이 될 것 같아서 제안드립니다!

Copy link
Author

Choose a reason for hiding this comment

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

확인 감사합니다! "자동으로"라는 표현이 2번 사용되어 한번만 반영하였습니다!

해당 파일에 대해서만 업데이트를 하고, 다시 컴포넌트를 렌더링할 것 입니다.
파일의 스타일, 렌더링 로직, 이벤트 핸들러, 혹은 effects 무엇이든 수정해도 됩니다.
- React 컴포넌트가 _아닌_ 파일을 수정하고 내보낸다면, Fast Refresh는
각 파일에 대해 다시 동작할 것이고, 나머지 파일들을 포함시킬 것 입니다. 그러므로
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
각 파일에 대해 다시 동작할 것이고, 나머지 파일들을 포함시킬 것 입니다. 그러므로
각 파일에 대해 다시 동작할 것이고, 나머지 파일들을 포함시킬 것입니다. 그러므로

애플리케이션을 다시 마운트할 것입니다.

앱에 [에러 바운더리(error boundaries)](https://react.dev/reference/react/Component#catching-rendering-errors-with-an-error-boundary)가
있다면(이는 프로덕션 환경에서 우아한 실패(graceful failures)를 위한 좋은 아이디어입니다),
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
있다면(이는 프로덕션 환경에서 우아한 실패(graceful failures)를 위한 좋은 아이디어입니다),
있다면(이는 프로덕션 환경에서 우아한 실패(graceful failures)를 위한 좋은 아이디어입니다).

Copy link
Author

Choose a reason for hiding this comment

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

여기서 사용된 ,는 앞의 "있다면~"이라는 표현 뒤에 붙은 ,로 사용되었습니다. 혼동을 발생시키는 것 같아 제거하였습니다.

이들의 의존성 목록이 무시됩니다.

예를 들어서, `useMemo(() => x * 2, [x])`를 `useMemo(() => x * 10, [x])`로
수정할 때, `x`(의존성)을 변경하지 않더라도 다시 실행될 것 입니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
수정할 때, `x`(의존성)을 변경하지 않더라도 다시 실행될 것 입니다.
수정할 때, `x`(의존성)을 변경하지 않더라도 다시 실행될 것입니다.

@xoxwgys56
Copy link
Author

@g2hhh2ee 리뷰 감사드립니다! 변경 적용이 많이 늦어졌습니다. 죄송합니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fast-refresh.mdx
2 participants