Single price grid component

Community feedback

• 

  Daniel 🛸•44,310

  @danielmrz-devPosted 11 months ago

  Hello @pohyongzhi!

  You did a very good job there!

  I have just one very simple suggestion for improvement:

  • Since the sign up button is a clickable element, it's nice to give the user a visual indication as well as a light color shift. Something like this:

  button:hover {
    background-color: /* color here */;
    cursor: pointer;
  }
  

  I hope it helps!

  Other than that, you did a great job!

  Marked as helpful

  1
• 

  raya•2,870

  @rayaattaPosted 11 months ago

  Hi Yong Zhi,congrats completing this challenge🎉.* A bit of advice you might be interested in. 1.every html document should have only one <h1></h1> element to indicate what the page is about then you continue off with the h1-h6 This is important for assistive technology such as screen readers to identify what the page is all about. 2.Your <div class="paragraph-group"> should actually be an <ul></ul> with list-style-type: hidden in css. This is because it indicates to assistive technology that it is a list not just a line up of divs. I also appreciate how much effort you put into making your html more semantic. Your solution is great

  Marked as helpful

  1

  ---

  

  Yong Zhi•110

  @pohyongzhiPosted 11 months ago

  @rayaatta Hi raya, thank you for the comment. I have amended my code! 😃

  1