The page on the monitor looks fairly decent. There are a few issues when looking at it on a smaller screen however. Primarily, the album sections:
I’d suggest making it so that at smaller sized screens the album and the title are all on top of each other instead of being side by side as this does feel a little cramped. It would be as simple as just setting the
property from row to column and maybe centering the content in the middle if you so choose.
Speaking of album sections, you don’t need to have so many different ones. Instead of having
album-4, replace them all with a single
album-container as all the properties are just repeated unnecessarily. It’s not important, but it is good practice and makes your code much easier to read and change.
Another good practice would be setting a margin for the main tag. Right now you have one margin for the left and one for the right, which is great if you ever decide you want your page to be off-center for one reason or another. But if you always want to keep it centered, it’s better to type
margin: 0 auto; and then set
width: 70%;. That way if you ever want to make it wider or thinner, you only have to change one property and not have to do math to figure out how to keep both sides even.
And that’s about all I got. The rest of it looks pretty good and there aren’t any glaring issues that I can see right now. Maybe someone else can pitch in. Keep it up