Skip to content

docs: fix markdown syntax typo in you-might-not-need-an-effect.md #1213

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

Merged
merged 2 commits into from
Jun 14, 2025

Conversation

moreal
Copy link
Contributor

@moreal moreal commented Jun 14, 2025

you-might-not-need-an-effect.md 문서에서 마크다운 문법 오류 수정

아래 문단을 보면 _이유_는 이라고 적힌 부분이 있는데 이는 렌더링 되어 보여질 때 <em>이유</em>는 으로 보여지는 것이 아닌, _이유_는 처럼 렌더링 되어 보여집니다.

image

analytics POST 요청은 Effect에 남아 있어야 합니다. analytics 이벤트를 전송하는 _이유_는 폼이 표시되었기 때문입니다. (개발 중에는 두 번 실행되지만 이를 처리하는 방법은 [여기](/learn/synchronizing-with-effects#sending-analytics)를 참조하세요.)

이는 CommonMark 스펙을 보았을 때 이어지는 단어 중간에 들어가는 경우 *을 사용해야만 강조가 적용되도록 제한되어 있는 것으로 보였습니다, https://spec.commonmark.org/0.31.2/#emphasis-and-strong-emphasis:

Many implementations have also restricted intraword emphasis to the * forms, to avoid unwanted emphasis in words containing internal underscores. (It is best practice to put these in code spans, but users often do not.)

그래서 *이유*는 꼴로 변경하는 것도 고려했으나 1) 개인적인 느낌에서 한글을 기울여 쓸 경우 옆 단어를 침범하여 보기 좋지 않다는 느낌이 있고1, 2) 다른 부분에서 의도적으로 띄어쓰는 방향으로 작성한 것으로 보이는 부분이 있어 _이유_ 는 꼴로 바꾸는 PR을 만들어 올립니다.

이러한 동작이 필요한 _이유__발생 시기_, 그리고 이러한 동작을 제어할 수 있는 _방법_ 을 살펴보겠습니다.

혹시 관련하여 이미 논의 및 결정된 사항이 있다면 공유주시면 참고해서 수정하겠습니다 🙏🏻

필수 확인 사항

선택 확인 사항

  • 번역 초안 작성Draft Translation
  • 리뷰 반영Resolve Reviews

Footnotes

  1. 관련하여 찾아봤던 논의를 첨부드립니다, https://github.com/mdn/translated-content/issues/1537

Copy link

vercel bot commented Jun 14, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
ko-legacy-reactjs-org ⬜️ Ignored (Inspect) Visit Preview Jun 14, 2025 9:11pm

Copy link
Collaborator

@lumirlumir lumirlumir left a comment

Choose a reason for hiding this comment

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

안녕하세요, PR 보내주셔서 감사드립니다!

아래 리뷰 남겨두었으니 참고 부탁드립니다🙇‍♂️

Copy link
Collaborator

@lumirlumir lumirlumir left a comment

Choose a reason for hiding this comment

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

LGTM 감사합니다!

Copy link

Size changes

📦 Next.js Bundle Analysis for undefined

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@lumirlumir lumirlumir merged commit f9788af into reactjs:main Jun 14, 2025
6 checks passed
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