Design comparison
SolutionDesign
Solution retrospective
I appreciate any feedback or suggestions you have regarding the solution.
Community feedback
- @vanzasetiaPosted over 1 year ago
Hi, Aayush Jaiswal! 👋
A couple of suggestions:
- Any words that are related to "image" (for example, picture, photo, logo, icon, graphic, and avatar) should not be included in alternative text.
- The alternative text of the logo should be the company name. In this case, it should be "Huddle".
- I think the "Try it Free" button is a link because once the users click that, they will get navigated to a sign-up page. The same goes for the "Get Started For Free" button.
- For your information, anchor tags are for navigation. The
<button>
element is for an action like opening a modal, submitting a form, toggling an element, etc. It is essential to use the correct elements. - It is recommended to have one
<h1>
element on a page. The<h1>
element is used to define the main heading of a page. Having multiple<h1>
elements can create confusion and make it difficult for screen readers and other assistive technologies to navigate the content. - Use
aria-label
attribute to label each social media link. Links must have accessible names so screen readers know how to pronounce them.
I recommend providing a non-minified CSS file for people to review. It will make it easier for people to give feedback on your CSS.
I hope my suggestions help you. Happy coding! 😄
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