-
Notifications
You must be signed in to change notification settings - Fork 145
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
my-happy.thoughts-webby #108
base: main
Are you sure you want to change the base?
Conversation
…dding component ThoughtItem,importedStyleForm.css, handle Form Submission
…y Thoughts box" This reverts commit ac3100e.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You've done a good job copying the design and your app works as expected so I will approve, but I don't understand why you have the components folder but don't mount any of your components? 👀
Please remove any unused code. For coming projects - split your project into components (as you started doing 👍 ).
@@ -1,3 +1,120 @@ | |||
import React, { useEffect, useState } from "react"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need to import React
<textarea | ||
value={newThought} | ||
onChange={(e) => setNewThought(e.target.value)} | ||
placeholder="Write your happy thought here..." | ||
rows="4" | ||
maxLength="140" | ||
/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remember labels when working with form elements
Netlify link
https://my-happy-thoughts-webby.netlify.app