Design comparison
Community feedback
- @VCaramesPosted about 1 year ago
Hey there! 👋 Here are some suggestions to help improve your code:
- Company logo's
alt
tag descriptions should NEVER include the keyword "logo"; It should only state the company's name.
- Remove all the
article
elements you added, as they are being used incorrectly and not needed for this challenge.
- Your
email input
is missing alabel
which will bevisually hidden
.
- The illustration is purely decorative, so it's better off to leave the
alt
description blank.
- The social media logos need to interactive. So they need to be wrapped inside an
anchor
element. They should also be built as anunordered list
.
- The JS for your
email input
validation needs work, asemail@[email protected]
is considered valid. You also want to add some sort of confirmation to valid emails as I wasn't sure if the validation was working.
If you have any questions or need further clarification, feel free to reach out to me.
Happy Coding! 👾
0@kevinfreitasv80Posted about 1 year ago- I do understand some points, but from my perspective (in this case), it's not necessary to include an
anchor
/a
element because it's not for a person/client; it's just a project that will be on my GitHub. But I do understand your suggestion, and I'll make the adjustment soon. - There's just one thing I didn't quite get – why should I remove my
article
? "The<article>
element in HTML is used to mark content that is independent and complete, like a logical unit that can be distributed or reused autonomously." - ChatGPT. It even mentioned that I could put my element inside adiv
, but from my perception, it wouldn't be as semantic. Nevertheless, I'll use adiv
because it's used for grouping structures, and thearticle
doesn't really seem that independent but more like a related element.
0@VCaramesPosted about 1 year ago@kevinfreitasv80
- Not a valid justification for neglecting the use of proper elements because failing to do so will only demonstrate a deficiency in knowledge.
Projects serve as a platform to showcase your expertise and skills to recruiters and fellow developers, providing them with insight into your proficiency. Employing incorrect elements will merely indicate to others that you lack essential coding skills.
- You are indeed correct in your understanding of the
article
element's purpose, but it's essential to note that the content you used it on doesn't align with that description.
Content designed for the
article
element can be used on ANY random website and make perfect sense regardless of its placement.A good example of this would be the QR code component challenge
More Info: https://www.w3.org/TR/2011/WD-html5-author-20110809/the-article-element.html
1 - Company logo's
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