Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

All comments

  • @fasih-mehmood

    Posted

    Flexbox is definitely the way to go when it comes to centering elements. Also avoid using ID's as much as you can and make use of classes instead. Avoid using absolute sizes as well, instead of pixels, try using % or rem or em.

    Other than that, the solution looks great! Keep up the good work.

    Marked as helpful

    1
  • @fasih-mehmood

    Posted

    Looking great! My only feedback would be not to use jQuery. It's really not needed anymore.

    Marked as helpful

    0
  • P
    Fluffy Kas 7,735

    @FluffyKas

    Submitted

    This challenge was my way to check if I improved in the past months and I'm happy to see I did! I completed it quickly and it was fun :) Still, if there's anything I could improve on (especially html wise), please let me know!

    Have a nice day everyone ^^

    @fasih-mehmood

    Posted

    Beautifully executed. The code is very clean and easy to understand which I really love. You did an awesome job!

    Keep up the good work and have a good day as well.

    2
  • @fasih-mehmood

    Posted

    Start by getting familiar with grid and flexbox.

    Marked as helpful

    2
  • @fasih-mehmood

    Posted

    Of course, but it'd be better to set the flex-wrap property to wrap as well while giving fixed size to the flex items so instead of getting squished on smaller screens, the individuals items wrap to the next row to maintain responsiveness.

    2