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
- @a1excpunkPosted over 2 years ago
- 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 helpful1 - Account deleted
Hello there! 👋
Congratulations on finishing your challenge! 🎉
I have some feedback on this solution:
-
Always Use Semantic HTML instead of
div
like<main>
<header>
<footer>
, etc for more info -
All Images must have alternate text
i hope this is helpful and goodluck!
1@DrMESAZIMPosted over 2 years ago@Old1337 please stop copying and pasting your comments to my wall.That is not helpful
0Account deleted@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 GitHubJoin 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