-
Notifications
You must be signed in to change notification settings - Fork 124
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
Week 7: Happy Thoughts Viktoria O. #63
base: main
Are you sure you want to change the base?
Conversation
const newMessage = event.target.value; | ||
setMessage(newMessage); | ||
setCharactersUsed(newMessage.length); | ||
if (newMessage.length > 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.
nice that you added a stretch goal!
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.
Hi! Your app looks just like the design and meets all the requirements and accessibility. Your code is very organized and easy to follow your thought process which makes it nice to read and review. I thought the number of components you have and the way it's mounted together was very clean! Nice job!
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.
Hey Viktoria great work! 💪💪
Loved to see the distribution off Data through the Thoughts.jsx
file, simple yet effective. What can I say but welcome to modern web development and II hope that you are enjoying the React framework. From my end it's seems that your grasping of the useEffect hook is pretty clear and you also implemented the date-fns lib. Great approach.
Netlify link
https://flourishing-mooncake-e332de.netlify.app/
Collaborators
Individual project