faq-accordion-card-main using Grid, FlexBox and javaScript

Community feedback

• 

  Muhammadh•1,140

  @Mr-jawPosted almost 2 years ago

  Hello there 👋

  Congratulations on completing the challenge

  HTML

  • Normally <div> tag is not used to display text, it is used to group some elements. Rather than using a <div> tag you could use a <p> tag. which will improve accessibility as well.

  • Use relative units such as em or rem for margin, padding, width, and height. avoid using px since it is an absolute unit.

  -It is a good practice to use CSS custom properties. learn more about it here

  • It will be more interactive if you made your cursor a pointer when hovering over the arrow icon. add these lines of code in your *style.css to achieve that

  .arrow:hover {
      cursor: pointer;
  }
  

  Other than that your solution is looking great 👍

  I hope this was useful 😊

  HAPPY CODING

  0