@Flashdaniel
Posted
Hi Satyam. Great work. There are few places I like to point out. You should not pust texts on <div>
and <span>
, because they are used for grouping element not for texts. Put your text in the right element tags like <p>
element tag can help in responsive design. Also you should know that screen readers don't read texts on <div>
and <span>
.
Make use of semantic HTML <time>
tag when working with time not <p>
tag. It include the datetime attribute to translate dates into machine-readable format, allowing for better search engine results or custom features such as reminders.
In your Stylesheet don't add both max-width and width at a time. Is either you are using one not the two or you can use min-width and max-width. When added Line height I saw that you forgot to add the unit. I recommend using responsive units like rem, em when working with font-size, margin, padding, width and use px when working with border and border radius . If responsive units was used In the project, you wouldn't need to use media query. Overall You solution is great!