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

HTML + CSS

Marcinβ€’ 570

@Frontmaniaac

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


I think everything works as it should

Community feedback

P
ApplePieGiraffeβ€’ 30,545

@ApplePieGiraffe

Posted

Hi, Marcin! πŸ‘‹

Well done on this challenge! πŸ‘ Your solution looks pretty good and is responsive! πŸ™Œ

I suggest,

  • Adding a little space between the heading and the top of the screen in the desktop layout (using margin/padding).
  • Using a single <h1> tag for the heading in this challenge since it's really a single heading (not two separate headings). You can easily style each part of the heading differently using the <span> tag.
  • Adding a max-width to the feature cards so that they aren't too wide when the layout first changes from desktop to mobile (and when the screen width increases in the desktop layout). πŸ˜‰

Keep coding (and happy coding, too)! 😁

Oh, and hey, if found these suggestions helpful, and upvote would be appreciated. ;)

1
Kyrylo Lvovβ€’ 160

@kyrylolvov

Posted

Hey! πŸ˜„

Great job on this challenge! πŸ†

There is a little issue with display of project due to margin of the blocks on 320px resolution. Everything except this is amazing! πŸ˜€

Good Luck and Happy New Year! πŸŽ„

1

Marcinβ€’ 570

@Frontmaniaac

Posted

@kyrylolvov Yup I fixed the min-width and works just fine :) Thanks for your comment and Happy new Year of 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