jaredly / jaredly/jaredly.github.io

React.arrayToElement -> ReasonReact.arrayToElement

Open
#8 0 comments 1 reaction 0 assignees View on GitHub
Dominant language
JavaScript
Stars
9
Forks
12
PR merge metrics
No merged PRs in 30d

Description

Excellent post @jaredly!

I noticed that in a few places the post has`React.arrayToElement` when it should say `ReasonReact.arrayToElement`.

Also, I got pretty tripped up here:

![image](https://user-images.githubusercontent.com/4427/27989572-4d92e430-63f0-11e7-8d4e-990e41afcdb6.png)

I didn't notice the `/* ... */` had code inside it! After all, it kinda looks like a comment. This led to me writing my state type definition after the call to `ReasonReact.statefulComponent`, which led to `Unbound record field items` errors that were pretty confusing for me.

Maybe that collapsable section should be open by default? Or maybe you could use a disclosure arrow instead of comment syntax.

Anyway, thanks again for writing this! 😄

Contributor guide

No contributing guide indexed for this repository

Research direction

Locate the post containing the `React.arrayToElement` references and the collapsed code example; the issue does not name a file or test. Correct the API name and make the hidden code easier to recognize or discover, then verify the rendered post presents the example clearly.

Written by the indexing model from the issue text.

Assessment

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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.