Hi Ridwan10000
Having gone through your project, here is my observation
HTML
- find out more about landmark elements, which are very important in the structural organisation of your site. For this challenge , I would include the
main
element as it identifies the primary content of the webpage. - the alt value of an image element should exclude some words such as
image, picture, logo etc
as these are automatically implied and will cause a repetition of words when using a screen reader
CSS
- you will need to reset your styles using a modern reset stylesheet
- this challenge does not need a lot of media queries , infact you can do it without any media queries. Instead of the explicit
width
on the containerwidth: 350px;
, usemax-width
- it allows your component to scale . Also for sizes , use rems in most cases instead of px
Marked as helpful
@Ridwan10000
Posted
Thanks for the suggestions @ChamuMutezva