Single price grid component

Community feedback

• 

  Dušan Lukić•1,660

  @dusanlukic404Posted about 3 years ago

  Hi Juveria, you did very nice job! I like your solution 😃

  There is a few things you should probably change:

  • My suggestion is to use button element instead of input with type of button and then to add cursor: pointer; and some transition to button
  • Your accessibility is good but you have one problem. You should increase heading element only by one. You have <h3> so use <h2> instead
  • Why Us section should be unordered list with list items for a way better accessibility instead of writing a paragraph with line breaks.

  There is a saying that you always look at everything to make it look good without using CSS. Then, you are 100% sure that accessibility is excellent and you can start styling your website. 🤩

  Marked as helpful

  0

  ---

  

  Juveria Dalvi•1,375

  @Juveria-DalviPosted about 3 years ago

  @dusanlukic404 thanks I've noticed that accessibility issue and I used margin-padding instead of line-break <br> tag

  0

  

  Dušan Lukić•1,660

  @dusanlukic404Posted about 3 years ago

  @JuveriaD yeah it is also common mistake. I don't suggest you to do in your next projects :)

  0

  

  Juveria Dalvi•1,375

  @Juveria-DalviPosted about 3 years ago

  @dusanlukic404 thanks I'm not going for same mistake again 😃

  0