geomydas• 1,060
@geomydas
Posted
After looking at your code, here is my feedback:
HTML
- Remove both the
article
andnav
tags. The article tags is often used in a news setting such as news websites, blog posts and etc. It makes no sense to use it here. The nav tag is used for internal links/pages and not external links such as linking to another website
CSS
- For me, the CSS seems to be following best practices but it is quite a bit hard to maintain. I suggest you take a look at the BEM methodology for this so you can name better classes inside the HTML
0