Mobile first layout using Flexbox

Community feedback

• 

  Adarsh Pratap•5,515

  @adarshcodesPosted over 4 years ago

  Hi! @Lizz Soup, your solution looks great👍. Let's see your solution with different-different aspects

  1. Design aspect

  • Your design looks nice.
  • You can change cursor: pointer; to the buttons on hover.
  • Decrease down the font-size of the heading inside the cards by a few px.

  2. Responsive aspect

  • Decrease the width of the images of cards at the breakpoint 857px around and also the bottom logo.
  • Overall it's fine

  3. Code perspective

  • Your code is clean and relative.
  • I like the way of commenting your code🙂.

  4. Issues aroused

  • There are HTML and Accessibility issues, as you already mention about it. You can try a few things:
  • In your font awesome script remove crossorigin="anonymous", it might works.
  • Check out this: https://fontawesome.com/how-to-use/on-the-web/other-topics/accessibility

  Great work on this solution🐱‍👤, keep learning and Happy Coding🐱‍💻

  0
• 

  Lizz Rennie•195

  @LizzSoupPosted over 4 years ago

  Not sure if anyone will see this - but I'd love some help fixing my font awesome icon errors

  0