Skip to content
OverviewSolutions
2
Comments
1
ANNE RUBIO
@AnneRubio

All comments

  • ANNE RUBIO•40
    @AnneRubioSubmitted almost 2 years ago

    This is my very first challenge on frontend mentor and I can't wait to start a new one :) I've had a lot of fun doing it. It helped me improve my knowledge of CSS grid as well as getting better on the responsive part.
    The most difficult element for me of this challenge was the navbar and more particularly the js code related to it: I'd be happy to learn if there's a better way to do it. Thank you and have a nice day

    Mobile responsive solution using CSS Grid and Flexbox

    2
    ANNE RUBIO•40
    @AnneRubioPosted almost 2 years ago

    Hello Geoff, Many thanks for your useful feedback. I will take the most of it to improve my current code :) On my side I've had a look at your code and live page which are great: kudos! I have a very minor feedback:

    • In the HTML, I believe the link with the script/js file is in general put at the end of the body for a better performance. This is not mandatory in a project like this one but rather a best practice to follow in general.
    • The hover and active elements seem to be missing Cheers
    0