Thanks so much for the feedback. MUCH appreciated. You weren’t too negative – very helpful to have someone looking at it with a critical eye.
I fixed the images in the information-boxes section on mobile (that was a late coding change when I finally realized they had provided images sized for mobile with the project), and you’re right about the mobile nav section. I had a weird padding issue I couldn’t get rid of until finally figuring out where it was coming from.
I was trying to get as close to spec as I could because this was my first coding project of any kind – there’s a couple of things on there that goes against every principle of design I’ve ever used.
Anyway, a couple of questions: When you mention tags (like setting the margin: 0; for tag, and when you talked about the number of rows for the main heading and paragraphs, I’m not sure what you mean.
On the footer, are you talking about adding more padding or spacing below the footer text, or something else? I could see how extra spacing below the text would be helpful. Didn’t realize til now, but the text does bump right up against the very bottom of the window.
I had some transitions coded in on my links, but they’re in theme with the black, gray and white of the site – and perhaps too subtle.
I checked yours out. Looks good! I stuck strictly with the spec on the shaded boxes for desktop, but if I did it over (and if it hadn’t been such a struggle the firs time), I’d have done it over the way you did with the shaded boxes on hover.
A couple of suggestions:
The font choice works really well for top-level display (the nav, and larger headers), but has a bit of a robotic feel for the smaller text. Might look better with a slightly different or closer to standard font for those spots, but that’s just one person’s opinion.
I’m interested in what led to you using the green for the hover color on the start button and course boxes, but not the information boxes. I think it’s just fine – just interested in the decision.
I like the text spacing out on the transition for the start here button in the main section at the top. I’d suggest either making that transition a bit slower or dropping the character/word spacing to, say, 0.2em instead of 0.3em. The transition itself is very cool. I think it’d look even better if the text wasn’t moving quite so far, so fast.
Only other thing: I like the way you set up the hover on the information boxes, especially when hovering over the middle box. When hovering over the top or the bottom box, I saw double lines between the two boxes that weren’t being selected, but didn’t see that when hovering/selecting the middle box. It’s a minor thing, certainly. Did you consider doing the same thing for those boxes on mobile? It looks very good the way it is. I’m just interested in how people make design and coding decisions, that’s all.
Hope that all made sense. Thanks again for helping me out!