Design comparison
SolutionDesign
Solution retrospective
Any feedback is greatly appreciated !
Community feedback
- @0xabdulPosted over 1 year ago
Hello
Jary Cruz
well congratulations on successfully completing the Easy bank landing page . A Few Feedback for improve your code ! and clear the Accessibility reportsSolution 📌
IN HTML 📃 :
LANDMARK 🚀
- The main landmark should be a top-level landmark. When a page contains nested document and/or application roles (e.g. typically through the use of iframe and frame elements), each document or application role may have one main landmark. If a page includes more than one main landmark, each should have a unique label.
- To Clear the Accessibility reports use the Semantic elements Or non - Semantic elements
- Note This Elements are don't sikp
- semantic elements :
<aside> , <artical> , <main>, <header> ,<section><footer>, <form> ect..
- non- semantic elements :
<div> , <span> ect ...
- for easy way to clear the Accessibility reports using non semantic elements Ex :
<body> <div class="container" role="main"> // All html code goes here : 📃 </div> </body>
- Or
- using semantic elements
- Ex :
<header> should be put heading or logo📸 </header> <nav> //Links here </nav> <main> Main of the contents 📃 </main> <footer> ©copy right here📍 </footer>
LINKS 🖇️
- link must have text only not use the img tag for easy way to clear the Accessibility reports
We use :
<a href="taxhike.html" aria-label="Read more about Seminole tax hike"> [Read more...]</a>
- Don't skip that !
I Hope you find the solution and it's useful comment for you And your Easy bank landing page project is Awesome🤩Finnally Happy Coding Developer
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