HTML, CSS grid, flexbox

Community feedback

• 

  Dušan Lukić•1,660

  @dusanlukic404Posted about 3 years ago

  Hi Jhuanjulio! I have a few suggestions for your solution:

  • It's better to set transition on hover on Sign Up button instead of using scale
  • Don't use <ul> for all elements. Example, for the top card you should just use heading and paragraph. Also, tags are there for making semantically good HTML, use <h1> not <strong> element
  • In Why Us section you should use <ul> element with a bunch of list items, not just paragraph with breaks
  • Per month text should be centered and have some opacity, not that gray color
  • It's not responsive so use media queries

  Marked as helpful

  0

  ---

  

  jhuanjulio•230

  @jhuanjulioPosted about 3 years ago

  @dusanlukic404 That's what I thought too at first. I just thought that maybe it's better to just use <ul> so I can manipulate the sizing/design more but anyways, I think I should've just kept it simple. Thankyou for your response!

  0