Responsive Meet Landing Page using Grid, Flexbox, Parcel, and Sass

Community feedback

• 

  P

  Dave•5,295

  @dwhenson

  Posted about 3 years ago

  Hi @nikoescobal - lovely job here! 🥳

  I think if you add the following rules to the footer you should get the result you want, I just tried it in the browser and it seems to work (perhaps you were missing the background-position part?):

      background-repeat: no-repeat;
      background-position: center top;  
      background-size: cover;
  

  Otherwise, One thing that's worth considering is whether an element is actually a button or a link . The most important thing is what the element will do not what it looks like. This page has a great summary and lots of useful links on this: https://css-tricks.com/buttons-vs-links/.

  I would think in this case the "download" could be a button, but the other "What is it" element should perhaps be a link? All this is debatable of course, the main thing is to be aware of the different use cases.

  Keep up the great work!

  Cheers 👋

  Dave

  11

  Marked as helpful