Made using HTML, CSS basics, Flexbox, Grid and mobile-first design

Community feedback

• 

  Dušan Lukić•1,660

  @dusanlukic404Posted about 3 years ago

  Hi Robert, well done! I have a few suggestions for your solution:

  • Make <h1> bigger on desktop version and also make sure to reduce margin-top on header on mobile version. It seems like 4.5rem would be perfect in my opinion
  • On mobile you should have centered text and that's fine but on desktop your articles should have default text-align (left)
  • Subscribe button on footer should be inline with input form

  For accessibility issues I recommend you to take a look on this link: https://developer.mozilla.org/en-US/docs/Web/Accessibility

  It helped me a lot so I hope that it would be helpful to you also 😄

  Marked as helpful

  0

  ---

  

  Robert Iancu•0

  @R0b3rtGPosted about 3 years ago

  @dusanlukic404 thank you for the feedback. I think I was in a hurry or something and I didn't notice the footer form not being as it should. The same with text size for desktop and alignment. I will try to fix some things. I really appreciate the feedback

  0