Submitted about 3 years ago
Created without any frameworks, just HTML, CSS and CSS grid.
@nmelgar
Design comparison
SolutionDesign
Solution retrospective
Any feedback is welcome
Community feedback
- @fidellimPosted about 3 years ago
Hi Nmelgar,
Great work on finishing this project! Looks responsive for both desktop and mobile view! Just a suggestion, you might be interested in adding active states on the buttons to add some effects like
button:active{ transform:scale(.95); }
I hope it helps :)
0 - @kens-visualsPosted about 3 years ago
Hey @nmelgar ππ»
I just have some quick tips to help you fix the accessibility issues
- In your markup,
<div class="component-container">...</div>
should be<main class="component-container">...</main> and <div class="attribution">...</div>
should be<footer class="attribution">...</footer>
. These will fix the accessibility issues. Don't forget to generate a new repot once you fix the issues.
I hope this was helpful π¨π»βπ» Cheers πΎ
0 - In your markup,
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