Landing Page using Flex-Box and Media Query's

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  Hello there, Nazim! 👋

  Great work on this challenge! Your solution looks pretty good! 😀

  Some feedback from me.

  • Always specify the type of the button. In this case, set the type of them as type="button". It's going to prevent the button from behaving unexpectedly.
  • "Cancel Order" button should be an interactive element (either button or anchor tag depending on what you think is going to happen after the user clicks the button).
  • Always use classes to reference all the elements that you want to style. Using id is going to make your stylesheet have high specificity (hard to maintain).
  • Create a custom :focus-visible styling to any interactive elements (button, links, input, textarea). This will make the users can navigate this website using keyboard (by using Tab key) easily.

  I hope this helps! Keep up the good work! 👍

  1

  ---

  

  Nazim Filzer•110

  @NazimFilzerPosted over 2 years ago

  @vanzasetia Tysm for the feedback

  0

  

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  @NazimFilzer No problem! 👍

  1
• 

  Kamasah-Dickson•5,590

  @Kamasah-DicksonPosted over 2 years ago

  Great job responsive on smaller devices Keep coding bro👍📌

  1

  ---

  

  Nazim Filzer•110

  @NazimFilzerPosted over 2 years ago

  @Kamasah-Dickson Tysm for ur Valuable Response. These words will be a Motivation for the nxt 🙂.

  0

  

  Kamasah-Dickson•5,590

  @Kamasah-DicksonPosted over 2 years ago

  @NazimFilzer Am glad to hear that 💪

  1