minor edit suggestions for React Foundations

Open Beginner friendly
#1,113 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
68/100
Issue type
Documentation
Clarity
Mostly clear
Activity status
Stale
Tech stack
react, typescript
Domain
documentation

Research direction

Use the Chapter 6 and Chapter 7 links in the issue as the entry points, then locate the source for their code samples in the repository. Remove the duplicated names.map() snippet, correct the Likes button text and spacing, and adjust highlighting so only the button line is highlighted; confirm both chapters render with the intended examples.

Written by the indexing model from the issue text.

Description

In Chapter 6 Displaying Data with Props (https://nextjs.org/learn/react-foundations/displaying-data-with-props), in the code sample where the array names is added, you should probably not have the names.map() code present because it's added and described in the next code sample.

In Chapter 7 Adding Interactivity with State (https://nextjs.org/learn/react-foundations/updating-state), in the code sample that follows "Then, you can check the initial state is working..." there are two issues:

  • The text for the <button> should be Likes ({likes}) (like the next code sample) instead of Like({likes}) ("Like" instead of "Likes", no space before parenthesis)
  • The // ... lines are highlighted, but I don't think that was the intention. It should probably just be the <button> line.
Dominant language
TypeScript
Stars
4.8k
Forks
2.2k
PR merge metrics
No merged PRs in 30d

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from vercel/next-learn

All issues in vercel/next-learn

Similar issues

More TypeScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.