TypeMaster Solution

Community feedback

• 

  Abdul Khaliq 🚀•72,560

  @0xabdulkhaliqPosted 8 months ago

  Hello there 👋. Congratulations on successfully completing the challenge! 🎉

  • I have a suggestion regarding your code that I believe will be of great interest to you.

  ---

  IMPROPER USAGE OF HEADINGS ↗️:

  • There's a minor semantic issue which is way more crucial and needs to be fixed, that's the usage of h3 for button like elements "PRE ORDER".

  ---

  • Actually those elements needs to be a button or a, for this challenge a is more likely to be good (because when the user clicks "PRE ORDER" they must want to go to form where they can preorder the stuff with their credentials)

  ---

  • And additionally add cursor: pointer CSS property for a elements, it's important for action elements because it changes the cursor from the default arrow to a pointer when hovering over the element. This provides a visual cue to the user that the element is clickable and encourages interaction. It also helps to provide feedback to the user by indicating which elements are clickable and which are not.

  ---

  • So you want to change the h3 into a element as shown in below,

  <a href="#" class="btn text-upper">pre-order now</h3>
  

  ---

  • Now your component's button has got the pointer & you learned about this property as well

  ---

  .

  I hope you find this helpful 😄 Above all, the solution you submitted is great !

  Happy coding!

  0