Skip to content
OverviewSolutions
8
Comments
3
Marwa Hammamouche
@MH-SunShine

All comments

  • Marwa Hammamouche•150
    @MH-SunShinePosted about 1 month ago

    Your solution is amazing, good job. I noticed you didn't set the alternate text for the image; this is important if the image cannot be displayed. keep going, you did a very good job.

    Marked as helpful
    0
  • Marwa Hammamouche•150
    @MH-SunShinePosted 2 months ago
    • Using semantic html tags to organize the solution was a clever idea, I really liked it.
    • In the CSS file, it's better to import the font family link. Also, specifying the flex direction will be much helpful. I recently learned that using max and min width instead of just using width property will help in the website responsiveness.
    • Please consider adding this in every single project you will work on: *{ padding: 0; margin: 0; box-sizing: border box; }

    You did a great job, keep going :)

    Marked as helpful
    0
  • Marwa Hammamouche•150
    @MH-SunShinePosted 2 months ago

    Thank you so much, I have read all the advices and I will make sure to follow them and learn more. Thank you

    1