Mobile First Page With Animations and Piano Sounds

Community feedback

• 

  dania•530

  @adimidaniaPosted over 3 years ago

  Hey!

  First, congratulations on completing this challenge. Keep the hard work up!

  Just a quick remark: In screens with a width between 1000px and 770px, some questions are not showing (overflowing), because you fixed the height of the card to 70vh with an overflow of hidden!

  0
• 

  T

  Chamu•13,130

  @ChamuMutezvaPosted over 3 years ago

  • you may have to implement a feature that allows users to turn of the sound effect if they wish to.
  • responsive image techniques are part of responsive web design and should be used for displaying images according to screen size. The methods includes use of srcset or picture element, and have advantages that includes saving your bandwidth making it cheap .The method you used below works but is expensive and wastes your bandwidth as each and every message is uploaded

  // NOT RECOMMENDED
   <div class="header">
  <img src="images/illustration-woman-online-mobile.svg" class="mobile" id="main-img" alt="Woman Online">
              <img src="images/bg-pattern-mobile.svg" id="back-img" class="mobile" alt="">
  <img src="images/illustration-woman-online-desktop.svg" class="desktop" id="main-img" alt="Woman Online">
              <img src="images/bg-pattern-desktop.svg" class="desktop" id="back-img" alt="">
          </div>
  

  • fix errors mentioned in the report

  0