Design comparison
Community feedback
- @AhmedMahroussPosted almost 2 years ago
Hey there! π Congrats on completing your first challenge!ππ»
Here are some suggestions to help improve your code:
FEM Best practice β , before moving on to the next challenge, always check your FEM report, to see what is incorrect and update your code with it so that you would not make the same mistake over again. This should be the first thing that should be done β οΈ right after submitting the challenge. Every site/component should always have β a main element to not only for semantic purposes but also help assistive technology find the main content of your content. For this challenge, it will serve as your components container β οΈ. More Info: π
MDN Main Element
Avoid skipping heading levels β οΈ. Always start with the h1 (which can only be used once) and you will go down the hierarchy level depending on the headingβs importance. To ensure that the "rating buttons" are fully accessible π―, they need to be built using a form β οΈ. Everything will be wrapped inside a fieldset which will have a legend that is visually hidden using CSS. Inside, there should be five input radios and each input should have a label attached to it to make the βratingsβ accessible. The last thing you will want to include will be a button so users can submit their choice. More Info: π
MDN: <input type="radio">
MDN: Field Set Element
If you have any questions or need further clarification, feel free to reach out to me.
Happy Coding! ππ
Marked as helpful0@Mohamed806HPosted almost 2 years ago@amahrous176 Thank you for the help I will try to be careful about duplicate mistakes and try to improve the codes I write next time
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