Creating styles using Sass

Community feedback

• 

  Ahmed Bayoumi•6,740

  @Bayoumi-devPosted over 2 years ago

  Hey Gideon Chidi, It looks good! but you just need to Contain the components in<main>...</main> tag.

  • All page content should be contained by landmarks, so the document should have one main landmark.

  <main id="container" class="flex">
        //...
     </main>
  

  • An img element must have an alt attribute, to provide alternative information for an image if a user for some reason cannot view it
  • I have a suggestion... Add a cursor: pointer to buttons, The cursor setting should inform users of the mouse operations that can be performed at the current location. I hope this is useful to you... Keep coding👍

  0

  ---

  

  Gideon Chidi•390

  @gideon-delPosted over 2 years ago

  @Bayoumi-dev Thanks I really appreciate it. I will do that right now

  1