HYDROCODER• 555
@HYDROCODER
Posted
Hey there! Good work. I would suggest the following if you don't mind:
-
Put some comments in your css. It will be helpful to you as well as for anyone viewing your code.
-
Do use ids, but rarely. ID's are very specific selectors and they can become messy when you have lot of elements in your HTML markup. For this challenge, I guess its fine but do be cautious about it. Use classes instead.
-
Never set fixed heights and widths for anything unless something very specific.
-
For the stats at the bottom, although it works, to be more semantic and meaningful, use ul and li's. Do learn more about semantic HTML to solve the accessibility issue.
Hope it helps :).
1
Shubham Sharma• 30
@S-A-M-0
Posted
@HYDROCODER that's awesome , thanks I appreciate your feedback :)
0
