Design comparison
Solution retrospective
Only HTML and CSS were used.
I didn't think this was difficult at all. I know my design isn't exactly the same and I am only missing cosmetic changes.
My only question, is there a better way to write my current code? Can I make my code more efficient?
Community feedback
- @0xabdulkhaliqPosted over 1 year ago
Hello there π. Congratulations on successfully completing the challenge! π
- I have other recommendations regarding your code that I believe will be of great interest to you.
HTML π·οΈ:
- This solution may cause accessibility errors due to lack of semantic markup, which causes lacking of landmark for a webpage and allows accessibility issues to screen readers, due to accessibility errors our website may not reach its intended audience, face legal consequences, and have poor search engine rankings, highlighting the importance of ensuring accessibility and avoiding errors.
- What is meant by landmark ?, They used to define major sections of your page instead of relying on generic elements like
<div>
or<span>
. They are use to provide a more precise detail of the structure of our webpage to the browser or screen readers
- For example:
- The
<main>
element should include all content directly related to the page's main idea, so there should only be one per page - The
<footer>
typically contains information about the author of the section, copyright data or links to related documents.
- The
- So resolve the issue by replacing the
<div id="container">
element with the proper semantic element<main>
in yourindex.html
file to improve accessibility and organization of your page
.
I hope you find this helpful π Above all, the solution you submitted is great !
Happy coding!
Marked as helpful1 - @TrenyceCodesPosted over 1 year ago
Good Job on creating and finishing your first frontend mentor post
The code you posted is good but here are some changes you can make to have it more readable.
- You can have one CSS file that is connected to your HTML file*
- You can add the root pseudo-class to your CSS to keep all your color hsl, hexes, or rgb well structured. You can check that out here css-root-pseudo-class.
- look at others' codes to get an understanding of the problems. Don't copy and paste because you won't learn properly
- if you are not using the HTML elements in your HTML file, don't add them to the CSS file. This can cause a confusing and big disorganizing mess.
- give your HTML elements ids, and classes, and use that in your CSS file.
I hope this feedback helps. Good job on finishing your first frontend mentor post
Marked as helpful1@DoubleOWebDevPosted over 1 year ago@Neice76
Thank you and will make the changes!
1@TrenyceCodesPosted over 1 year ago@DoubleOWebDev youβre welcome. Good job on your project.
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