HTML + CSS

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted almost 4 years ago

  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

  @kyrylolvovPosted almost 4 years ago

  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

  @FrontmaniaacPosted almost 4 years ago

  @kyrylolvov Yup I fixed the min-width and works just fine :) Thanks for your comment and Happy new Year of coding!

  0