Responsive landing page using CSS GRID and mobile-first method

Community feedback

• 

  Lucas 👾•104,400

  @correlucasPosted over 2 years ago

  Wow Garang, congratulations for this solution.

  You did good, I saw the live site mobile and desktop, the mobile version is perfect, there few things you can fix for the desktop version. Note that the background is repeating, fix that applying background-repeat: no-repeat; and background-size: cover; to contain the background inside all the body fitting all the screen. Maybe you can set one or two media queries before the main mobile queries, to do some small adjustments to your solution, maybe changing the heading and paragraph sizes.

  Keep going bro, you've improved since the first challenge I saw. 👏👏👏

  Marked as helpful

  1

  ---

  

  Garang Deng•190

  @garang-dengPosted over 2 years ago

  @correlucas thanks bro, you always give me the energy to keep on pushing ❣️💯

  Can I fix the background-repeat on the desktop only?

  0

  

  Garang Deng•190

  @garang-dengPosted over 2 years ago

  Hey Lucas, could you please check my code and tell me where I went wrong. That created HTML issues on the report 🤗

  Best regards

  0

  

  Lucas 👾•104,400

  @correlucasPosted over 2 years ago

  @Garang-Deng The error that's showing in the report is related to a web-kit property but looking your css I don't see any of these properties, did you use something different in your css? Try to update the report and see if still displaying that.

  1

  

  Garang Deng•190

  @garang-dengPosted over 2 years ago

  @correlucas I've not used those properties in my CSS I think they were generated by GitHub pages. The report still shows them even after the update 🥺

  0

  

  Lucas 👾•104,400

  @correlucasPosted over 2 years ago

  @Garang-Deng I'm sorry for that bro, even if you go to the css sheet and use ctrl + f you didn't find those properties? Consider use vercel.com for live pages its much easier and simple to setup than Github Pages.

  Marked as helpful

  0