Hi there!
I’ve just started with freeCodeCamp, and am working through my Responsive Web Design Projects. I’ve just completed my tribute page project - based on what I have studied so far. (Plan is to eventually expand it as I learn more).
Does anyone have any feedback or constructive criticism? My end goal is to get a job as a front-end web developer - so any feedback helps!
https://codepen.io/Pandoraa/pen/VOKZoo
Your page is responsive, and your HTML has no warnings from codepen, so that’s a great start. Your purple-on-purple links however are quite hard to see, as they have very low contrast.
Looking at your HTML, I noticed you use divs with classes like “one” , “two”, and “three”. Class names ought to be as descriptive as possible, and even better than that is to use semantic elements. How about instead of a div with a class of “one”, using the <header>
tag instead? You could do similar with <footer>
for the .four class, and there’s probably similar tags (such as <nav>
and <section>
) you could use for the other classes.
3 Likes
@chuckadams
Thank you very much! I originally named the classes ‘one’, ‘two’, etc when setting up the layout of my grid and totally forgot to adjust them! I will look more into the CSS specific tags more now!
I checked all my colours with a contrast checker and then did a last minute move/colour adjust at the end - totally forgot to replug it back into the contrast checker!
Thank you so very much for your feedback, I will apply those today!