Single price grid component

Community feedback

• 

  P

  Dave•5,295

  @dwhensonPosted almost 2 years ago

  Hey @CYCHAN00 👋

  Lovely job here, and not much to comment on 🙌, but here's a couple of things to think about:

  • Button vs link? 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 imagine clicking on 'sign up' would take me to a new page, if you also think that then it should be a link.

  • I'd also get rid of all those br tags inside the square on the bottom right and just put everything inside a ul and make each item an li. I think this would be more semantic as they seem to be a list to me. You will need to remove the default list styling, but once that's done things will appear as they are now, but be better for people using assistive tech to access your site.

  Otherwise, keep up the good work and great job! 🥳

  Cheers Dave

  0