@Islandstone89
Posted
HTML:
-
The image has meaning, so it must have proper alt text. Write something short and descriptive, without including words like "image" or "photo". Screen readers start announcing images with "image", so an alt text of "image of Jessica Randall" would be read like this: "image, image of Jessica Randall".
-
Headings must always be in order, so you should never go from a
<h2>straight to a<h5>. Also, "London, United Kingdom" should be a<p>. -
The social links are not headings. Change them into links.
CSS:
-
Performance-wise, it's better to link fonts in the
<head>of the HTML than using@import. -
It's good practice to include a CSS Reset at the top.
-
max-widthon the card should be in rem. -
Change the
font-sizeon.aboutfrompxto rem.font-sizemust never be in px. This is bad for accessibility, as it prevents the font size from scaling with the user's default setting in the browser. -
Since all the text has the same font, you should place the
font-familyon thebody, and remove it elsewhere.
@seshadrianadasu
Posted
Thanks for remember my faults and correction
@seshadrianadasu
Posted
Thanks for remember my faults and correction
