Skip to content
OverviewSolutions
13
Comments
2

Nassim

@Nassim555Morocco200 points

I’m a mysterious individual who has yet to fill out my bio. One thing’s for certain: I love writing front-end code!

I’m currently learning...

typescript, sass, webpack

Latest solutions

View more solutions

Latest comments

  • Nassim•200
    @Nassim555

    Hi Mark! Congratulations on your solution , I suggest to wrap your content in a main element and I think you forgot to add alt text to your images and I also suggest to to use h1 instead of h3 for the cards title as it's the first and only heading on your page all of these changes are necessary for improving accessibility there's a section on your frontend-mentor's solution page to check accessibility and Html issues . Happy coding.

    Marked as helpful
  • Nassim•200
    @Nassim555

    Hi , great solution I love your approach especially when u decided to add the dark and light mode to spice it up , one thing though it's preferred to cursor:pointer; to show that the toggle is a clickable element.

    Marked as helpful