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 | FLEXBOX

Dwightβ€’ 215

@SarsiPC

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


10/17/20

Honest feedback/criticism is appreciated! Any suggestions on how I can improve this solution?

Update 10/17/20

  • Cleaned & tweaked CSS code

Update 10/19/20

  • Lighter shadows
  • Fixed the fonts
  • Added <br> in-between the words 'data' & 'points' in the paragraph line

Community feedback

Randeep Ranaβ€’ 45

@Randeep-Rana-au8

Posted

Your Solution is Good, one thing that you can do is just give some margin-top to the heading, make the first line of heading thinner because it looks slightly bold then the question, and add <br/> tag in paragraph line to make it in two lines...

But it overall looks good. Good Job

2

Dwightβ€’ 215

@SarsiPC

Posted

@Randeep-Rana-au8 pssst, I updated it! I'm such a noob. Didn't know I had to declare the font properties by copy & pasting: <link href="https://fonts.googleapis.com/css2?family=Poppins:wght@200;400;600&display=swap" rel="stylesheet">

Thanks for your suggestions!

0
Randeep Ranaβ€’ 45

@Randeep-Rana-au8

Posted

@SarsiPC Everyone is a noob in the starting man just give some time to yourself one day you will become a proπŸ”₯

1
P
ApplePieGiraffeβ€’ 30,545

@ApplePieGiraffe

Posted

Hey, good job, Dwight! πŸ‘

Your solution looks good and responds well! πŸ‘

I only suggest,

  • Setting a max-width on the feature boxes in the mobile layout of the page (so that they don't look too wide when the layout of the page first changes from desktop to mobile).

Keep coding (and happy coding, too)! πŸ˜„

1

Dwightβ€’ 215

@SarsiPC

Posted

@ApplePieGiraffe Well, spanning the boxes all the way through was my intention in the first place. I think it looks good, but I updated it to make it less "stretchy", I guess.

Thanks for your suggestions!

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