Looking to hire developers?
Hello all! This was fun.
I tried to hit all the bonus challenges listed in the project description:
localStorage to save the current state in the browser that persists when the browser is refreshed.createdAt strings from the data.json file, try using timestamps and dynamically track the time since the comment or reply was posted.currentUser object from within the data.json file.Thanks!
@anoshaahmed
Posted
To get rid of the accessibility/HTML issues shown in your Report:
<h1> in your codealt="" to your <img> tags<button> should have an aria-label ... Click here to read more<input> should have corresponding <label>Great job! :)
Marked as helpful
@BinarySenseiii
Posted
everything is good mate but one problem i found is upvoting user can vote only one time whether it's increment or decrement, in your solution user can increment infinite which make no sense
@BenjaDotMin
Posted
@BloodySteps Hey! Thank you so much for your feedback. The ability to continually up/down vote the comments are to show the re-ordering of the comments based on their scores. As this is based on localStorage, if I was to limit it to one vote per person, there would be no way to show the ordering functionality (although the inner reply order is based on date created, as per the brief). But in a "real world app" you are absolutely right, that would make no sense.
I appreciate the kind words!
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord
