SimpleForm

[biglovisa]

For your consideration, @wfro :zap:

A component that submits a value. Not super interesting, but the next exercise could be a more complex one with multiple fields with validations.

solution

[wfro]

Hey! I think this is fantastic so far. The specs themselves read very clearly and have really nice incremental progressions. I left a few comments around testing strategies and a few other random thoughts.

Here are a few more in no particular order:

• Nitpick: SimpleAddNewForm could potentially be simplified to SimpleForm
• I wonder if it would be good to fill in prop types for users? We have to render these things in test so we necessarily have to make decisions about props. They'd be easy to ignore as well, so we can set an example by being consistent about them.
• I think putting the test files directly in the exercise directories (e.g. src/03/SimpleAddNewForm-test.js or src/03/__tests__/SimpleAddNewForm-test.js ) helps the exercises feel like a "unit". We can have users run each test file directly, meaning they won't have to worry about re-xiting tests after completing exercises when using something like npm test
• The form could be more accessible, e.g. using a form tag and adding a label for the text input (https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/forms/Basic_form_hints)

I'm happy to help with any of these :star: 😄

[biglovisa]

• I like the SimpleForm name
• Hm... my first thought is that we shouldn't fill in the prop types since they are included in the test. We could add a better README that goes through the skeleton code in each example and explain more in detail what the prop types are. What do you think?
• Yes for colocation 🏠
• Yes! A11y all the way 💯

[wfro]

:ship: