Design comparison
Solution retrospective
I had a bit of trouble getting the 3 images at the bottom to be the same size. I ended up setting the size of them individually and getting them close, but I assume there is a better way.
Community feedback
- @AdrianoEscarabotePosted almost 2 years ago
Hello Cory, how are you? I truly loved your project's outcome, however I have some advice that I hope you'll find useful:
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 solve this problem do this:
<a href="/" class="bg-[#F0604F] text-white tracking-widest py-1.5 px-6 mt-3 mb-6 hover:bg-[#00011A]">READ MORE</a>
You could have used a h1 to make the project more accessible. Every page must have a level 1 header that identifies the primary title and follows the h1-h5 format for screen readers.
<h1 class="text-5xl font-black">The Bright Future of Web 3.0?</h1>
The remainder is excellent.
I hope it's useful. 👍
Marked as helpful0 - @KarrimorRiflePosted almost 2 years ago
When working on project like these its best practice to categorise each section into main, article, nav etc using divs, main, etc., its much cleaner to read and much easier to organise.
For making the images the same size I think it would be more beneficial to incorporate some of your own CSS. These tools are fantastic but sometimes writing your own CSS to sort a problem like that will be much faster, thats IMO though
Marked as helpful0 - @catherineisonlinePosted almost 2 years ago
For the background you can use rgba(254, 253, 249, 255) so it matches the design
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