Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Four Card Feature Section Master

P

@kaamiik

Desktop design screenshot for the Four card feature section coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


What are you most proud of, and what would you do differently next time?

.

What challenges did you encounter, and how did you overcome them?

.

What specific areas of your project would you like help with?

.

Community feedback

P
Daniele 300

@dedo-dev

Posted

Hi Kamran, well done to solve this challenge. I'm here to give you some suggestions.

for HTML

  • The main heading doesn't need to be split in an h1 and h2, you can simply wrap this part " Powered by Technology " in a b tag,
  • Don't use section for the cards, instead you can opt for article or div;
  • Icons in the cards are decorative elements so you can probably leave alt text blank;
  • I think the main tag should be used to wrap all the content, except for the footer of course;

for CSS

  • Are you sure that this massive use of calc doesn't cause any problems with performance? I ask you because I never seen something like that before;

Hoping you find this useful

Cheers ✌️

Keep Learning 📖 Keep Coding 💻
0

Please log in to post a comment

Log in with GitHub
Discord logo

Join 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