Submitted over 1 year ago
challenge solved successfully using some HTML & css , check it out
@Ayakhaled0197
Design comparison
SolutionDesign
Community feedback
- @tylerhyndman484Posted 2 months ago
Hello!
You have the right idea here but a couple of things I can suggest to improve:
- Need to add some spacing at the top of page above the title text
- Add some spacing below the sub "Powered by Technology" text
- Add a little more spacing below the "Our artificial" text
- The cards need to have more padding around the top, left and right side
- The cards need to have less padding on the bottom of the cards underneath the image
- The cards need to be smaller but fixing a lot of the issues about and below should help with that
- The text needs to be bigger in general across the board. Start with the 15px baseline and then increase everything else accordingly that seems to need to be bigger
The gap on the grid and the layout look great though! Nice job on that!
I hope this helps! Let me know if you have any questions!
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