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

Nitpicks for sections 2-6 #242

Merged
merged 4 commits into from
Oct 24, 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
2 changes: 1 addition & 1 deletion exercises/02.side-effects/01.problem.effects/README.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ full refresh of the page when the user clicks the back/forward button.
🧝‍♂️ I added a new utility called `setGlobalSearchParams` which allows us to set
the URL search params without triggering a full-page refresh. So whenever the
user clicks "submit," we update the URL search params. The trouble is, when they
hit the back button, the search doesn't stay synchronoized with the URL (yet).
hit the back button, the search doesn't stay synchronized with the URL (yet).

You can take a look at <DiffLink app1={-1} app2={0}>my changes</DiffLink> for
details on what I did.
Expand Down
6 changes: 3 additions & 3 deletions exercises/02.side-effects/README.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -25,11 +25,11 @@ useEffect(() => {
])
```

`useState` is for managing our react component state and `useEffect` is for
managing side-effects. Side-effects are things that happen outside our react
`useState` is for managing our React component state and `useEffect` is for
managing side-effects. Side-effects are things that happen outside our React
components.

For example, things outside our react components include:
For example, things outside our React components include:

- Browser APIs like local storage, geolocation, media devices, etc.
- Integrations with third-party libraries like D3, Chart.js, etc.
Expand Down
2 changes: 1 addition & 1 deletion exercises/03.lifting-state/01.problem.lift/README.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ to know what I've done. I pretty much just moved stuff into separate components
to get a little more organization in here.

<callout-info>
🦉 check out [When to break up a component into multiple
🦉 Check out [When to break up a component into multiple
components](https://kentcdodds.com/blog/when-to-break-up-a-component-into-multiple-components).
</callout-info>

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ const defaultState = Array(9).fill(null)
// 🐨 create a variable for the key you'll use for storing the squares
// 💰 'squares' should work well.
function Board() {
// 🐨 use the callback form for useState the callback should:
// 🐨 use the callback form for useState. The callback should:
// 1. get the value from localStorage using the key you created above
// 2. parse the JSON from that value
// 3. return the parsed value (or the default value if there isn't one)
Expand Down