HTML, CSS

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted about 4 years ago

  Hey, nice work, Marawan Mohamed! 👍

  Your solution looks good! 🙌

  I suggest,

  • Allowing the background SVG to fill up the entire area of the screen in the desktop layout. You should be able to accomplish this by setting its background-size property to cover.
  • Adding a little space between the image and the text content of the page in the desktop layout so that there's still some space when the screen width decreases.

  Keep coding (and happy coding, too)! 😁

  2

  ---

  

  Marawan Mohamed•90

  @marawanmohamed9876Posted about 4 years ago

  @ApplePieGiraffe Thank you, that's helped me a lot

  0
• 

  Griselda Knight•280

  @Greeshma2903Posted about 4 years ago

  the site is quite responsive! Good one. Keep going! All the best!

  1

  ---

  

  Marawan Mohamed•90

  @marawanmohamed9876Posted about 4 years ago

  @Greeshma2903 Thank you

  0
• 

  Seck Mamadou•325

  @doums85Posted about 4 years ago

  Hi Mohamed,

  Very good work ! 👍

  Keep coding you're good 😊

  1

  ---

  

  Marawan Mohamed•90

  @marawanmohamed9876Posted about 4 years ago

  @doums85 Thank you 😊

  0