Simple card using HTML,CSS (Flex-box).

Community feedback

• 

  Fat•910

  @FahatmahPosted over 2 years ago

  Great work there!

  I just see in your report that you have some accessibility issues. Main issue is not using semantic HTML which are main, section, footer and this will avoid the issues in your code. Although you used section tag, it shoud have the main tag first. But nice job done there. Hope this helps you.

  Marked as helpful

  1

  ---

  

  satya surndra•180

  @satya-hashPosted over 2 years ago

  @Fahatmah Thank you for the feedback. And I try my best to resolve the issues and improve my skills.

  1
• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  Hello, @satya-hash! 👋

  Good effort on this challenge! 👍

  Some recommendations from me.

  • Always specify the type of the button. In this case, set the type of them as type="button". It will prevent the button from behaving unexpectedly (like submitting).
  • The value for the href attribute of <a href="*">Change</a> is visiting to "Page Not Found". I would recommend using a number sign or forward slash instead.
  • There should not be text in span and div alone whenever possible. Instead, wrap the text with a meaningful element like a paragraph element.

  That's it! I hope you find this useful! 😊

  Marked as helpful

  0

  ---

  

  satya surndra•180

  @satya-hashPosted over 2 years ago

  @vanzasetia Thank you for your help and support. I will try my best to correct my mistakes and improve my skills 🙂.

  0

  

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  @satya-hash Happy to help! 😊

  0