A1ex• 500
@a1excpunk
Posted
- some parts are missing, like - the word "advice", hashtag, quotes, divider lines.
- button doesn't call the method to fetch API, so there is only one call, on page load and that's it.
Marked as helpful
1
Design comparison
Solution retrospective
All feedback is welcome as well as comments on my YouTube video
YouTube video link: https://www.youtube.com/watch?v=4kwmzLxRoKI
Community feedback
IRVINE MESA• 1,855
@DrMESAZIM
Posted
thanks @a1excpunk very helpful indeed.
1
Old• 6,240
@Old1337
Posted
Hello there! 👋
Congratulations on finishing your challenge! 🎉
I have some feedback on this solution:
-
Always Use Semantic HTML instead of
divlike<main><header><footer>, etc for more info -
All Images must have alternate text
i hope this is helpful and goodluck!
1
IRVINE MESA• 1,855
@DrMESAZIM
Posted
@Old1337 please stop copying and pasting your comments to my wall.That is not helpful
0
Old• 6,240
@Old1337
Posted
@DrMESAZIM its a common issue that i find on almost every solution so i made a text for it instead of re-writing it everyday, you have this issue and you asked for feedback thats why i commented. anyway i am sorry for trying to help..
0
Please log in to post a comment
Log in with GitHub
Join our Discord community
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