single-price-grid-component

Community feedback

• 

  Laian de Souza Braum•0

  @laianbraumPosted almost 4 years ago

  I agree with phonzdev, you should definitely use <ul> for more semantic meaning, but you could also use just <br> in this one.

  Also, I noticed that the "Sign Up" button still has the inherit border of browser, to avoid that, use border: none. The button appearance will be definitely way better. Nice solution.

  0

  ---

  

  Mojtaba Mosavi•3,760

  @MojtabaMosaviPosted almost 4 years ago

  @laianbraum I truely appreciate the feedback, thank you.

  0
• 

  Alphonzo Escolar•120

  @phonzdevPosted almost 4 years ago

  Hi @MojtabaMosavi, love the responsiveness on this one!

  Here are a few things I'd recommend reviewing after taking a look at your code:

  -use wrapper to contain the whole three boxes and set a max-width on it so it doesn't expand to the whole width of the browser

  -on the "Why Us" section, I'd recommend using the <ul> tag to make it more meaningful :)

  Keep up the good work! Cheers!

  0

  ---

  

  Mojtaba Mosavi•3,760

  @MojtabaMosaviPosted almost 4 years ago

  @phonzdev tanks a lot for the feedback.

  0