News homepage solution - Responsive layout using CSS Grid
Design comparison
Solution retrospective
- How do set <img> sizes relatively to maintain correct image sizes in mobile/desktop view?
Community feedback
- @AdrianoEscarabotePosted almost 2 years ago
Hi Sai Prasad Padhy, how are you? I really liked the result of your project, but I have some tips that I think you will enjoy:
I noticed that you used a
button
in which case the best option would be ana
, because in my head when a person clicks on a button written Read more, he is not confirming a form, or something like, it will be redirected to another page, to read more about!To resolve do this:
<a href="/" class="btn-read-more">Read more</a>
We have to make sure that all the content is contained in a reference region, designated with HTML5 reference elements.
HTML5 reference elements:
<body> <header>This is the header</header> <nav>This is the nav</nav> <main>This is the main</main> <footer>This is the footer</footer> </body>
The rest is great!
I hope it helps... 👍
Marked as helpful0@PadhysaiPosted almost 2 years agoThanks, @AdrianoEscarabote for your suggestions. I will follow them next time.
1
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