Responsive landing page using Flex

Community feedback

• 

  Phantomxxs•20

  @sharonolatPosted over 2 years ago

  Hello

  1. First you should probably use rem units instead of em
  2. You could put the icon and text inside the button tag set the width 100% and style them together that stacking them, using z index and negative margin. Would save you lines of code
  3. You could also try to use more explanatory class or id namings. classes of 'One' 'two' and all you used are a little vague

  1

  ---

  

  Eladies•40

  @gift121Posted over 2 years ago

  @sharonolat Thanks so much, I really appreciate your feedback, will work on it. Please can you point out some books and videos I can use to learn JavaScript as a beginner

  0