LandingpagewithCSS+HTML

Community feedback

• 

  Maria•290

  @itsfedorukPosted almost 4 years ago

  Hi Abraham, well done on your first challenge ☺️ To add on previous comments about changing the container property of height: 100vh, I'd also suggest moving two circles from HTML images to CSS as background images for body. It'll remove extra overflow, and you'll have more control over styling and positioning of circles as well.

  Hope it helps, and enjoy your day 🙌

  1

  ---

  

  Abraham Gutierrez•25

  @abgutierrez507Posted almost 4 years ago

  @hellomerope thanks so much for your Feedback, extremely appreciated :)

  0
• 

  Abraham Gutierrez•25

  @abgutierrez507Posted almost 4 years ago

  Best feedback ever!

  0
• 

  Fauzan Reza•180

  @codebyfauzanPosted almost 4 years ago

  Good effort Abraham. I think you should change height properties of .container from 150vh to 100vh and position: auto to position: relative. Then try to change bottom or left properties from bg-ballon-bottom. It should help

  0
• 

  Gabriel Schiferdecke•55

  @gschiferPosted almost 4 years ago

  Why did you not center the card in the project? Did you have difficulties with this approach?

  0