@kem522
Posted
Hello there!
This looks good! As for your question about background images being in the HTML or CSS, generally all styling should be in the CSS unless for some reason it can't be (e.g images being dynamically added via JS) so the way you have done background images here is correct.
Another couple of bits and pieces:
- It's great that you've added alt text to your images, but you don't need to add the word 'pic' as it's already assumed your alt text is referring to an image.
- When using 0 values in your CSS you don't need to add px, em or rem at the end e.g
margin: 0
is just fine!
@KtGitIt
Posted
Hi there!
Thanks for reply and suggestions. After reading your both advice, now I can recall the training I had about it. It does ring a bell now :)