@AlinggggSubmitted about 2 months ago
What are you most proud of, and what would you do differently next time?
Implemented media query for the mobile version. Also used sass but only the nesting for the most part.
manasmaity506
@manasmaity506All comments
- @manasmaity506Posted about 2 months ago
Great work @Alingggg . The website looks great.
- You can improve your solution by using sematic html elements.
- You can use picture element for rendering different images for different screen sizes.
0 - @RafiMauludi255Submitted 2 months ago@manasmaity506Posted 2 months ago
Your solution can be improved using semantic elements like section, header and main. The code seems good and readable. But The layout of the page breaks in small screen sizes. You should look at it and fix your media query styles.
0 - @LimnosaSubmitted 3 months ago@manasmaity506Posted 2 months ago
Great work @Limnosa . A little suggestion , you can use the img tag to get the card image, the code in index.html will look more readable that way.
0 - @hadodevSubmitted 2 months ago@manasmaity506Posted 2 months ago
Great work bro. But you can remove the footer section. That is not needed in this solution. But I guess, you have done it consciously, looks great.
0
