Latest solutions
Latest comments
- @JuanAntonioGarciaCantuSubmitted 2 months ago@Tamir-NahmiasPosted about 2 months ago
your web has met most of the criteria , way to go! your code is simple and easy to read, useable and user friendly. only the interactive tabs were missing. all in all , you did great
0 - @MohammadShoolSubmitted about 2 months ago
- @S44r4hSubmitted about 2 months ago@Tamir-NahmiasPosted about 2 months ago
overall , your website looks great! very similar to the original design we were given . nonetheless ,in your next project, consider adding semantic HTML elemetns inside the body tag such as "main" and "footer" (for the card content , and the "Attribution" section correspondigly), they are excellent for accesibilty and for SEO ranking. also, for non semantic elements such as div, try to add roles . the layout of the screen is just fantastic, and corresponing very well to diffrent sizes , altough I didn't see min size for mobile phones. your code is very well written , you succeded your task with minimum div elements and brought fantastic results! (btw, I've learnt a thing or two from your desgin , thanks!)
Marked as helpful1