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

"You Might Not Need an Effect" の翻訳を変更 #733

Merged
merged 2 commits into from
Feb 13, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions src/content/learn/you-might-not-need-an-effect.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
---
title: 'エフェクトは必要ないかもしれない'
title: 'そのエフェクトは不要かも'
---

<Intro>
Expand Down Expand Up @@ -360,7 +360,7 @@ function Form() {

イベントハンドラとエフェクトのどちらにロジックを入れるべきか選択する際には、ユーザの観点から*それがどのようなロジックなのか*を自問自答するようにしましょう。そのロジックが特定のユーザ操作によって引き起こされる場合は、イベントハンドラに保持します。ユーザが画面上でコンポーネントを*見る*ことによって引き起こされる場合は、エフェクトに保持します。

### 計算の連鎖 {/*chains-of-computations*/}
### 計算の数珠繋ぎ {/*chains-of-computations*/}

時々、他の state に基づいて state の一部を調整するエフェクトを連結させたくなることがあります。

Expand Down
2 changes: 1 addition & 1 deletion src/sidebarLearn.json
Original file line number Diff line number Diff line change
Expand Up @@ -181,7 +181,7 @@
"path": "/learn/synchronizing-with-effects"
},
{
"title": "エフェクトは必要ないかもしれない",
"title": "そのエフェクトは不要かも",
"path": "/learn/you-might-not-need-an-effect"
},
{
Expand Down
Loading