@bbsmooth thanks for taking the time to look over my project
You’re absolutely right about the scroll bars - I will definately fix that.
TBH I need to work on my understanding of responsiveness as a priority - I’m finding CSS much more complicated than HTML, but I’m taking steps to get better!
Thanks for this - I didn’t fully comprehend the docs when I looked at captioning, however, just for the purposes of this excercise, I’m going to leave this one as a
<div>, as the brief does explicitly say to use one. I will remove the
<figcaption> element and just use a
I think I did that because the brief doesn’t explicitly tell us to use one; only that we should use “an element with a corresponding
id="main"”, but now I realise I probably should have read between the lines on that one!
After a bit of browsing, I can see there are definately some advantages of using the
<time> tag for this, however, it would also be a valid
timedate atrribute value within the
<time> tag, would it not? So should I just use
From what I can see, the
datetime attribute is used for calendar integration, and so wouldn’t be much use here
…as for the
<strong> tags, I just used those for styling, but I’m getting the impression I should try to keep my styling within my CSS as much as possible, rather than in my HTML, right?
Yes, a change of mindset is needed I think! I need to be thinking more semantically when I write - thank you.
You can just pretend you never saw that
OK, I’m re-writing the whole thing, taking into account all your feedback!
I’ve started using a proper text editor rather than Codepen, too (because writing in VSCode vs writing in Codepen = no contest!). I will post the ‘refactored’ version as soon as I’m happy with it!