nanoanno / teacozy Goto Github PK

Seen here: https://github.com/nanoanno/teacozy/blob/master/resources/css/style.css#L27

Very clear and nicely done!

Seen here: https://github.com/nanoanno/teacozy/blob/master/index.html#L52

This was nicely done! :)

Seen here: https://github.com/nanoanno/teacozy/blob/master/index.html#L16-L18

I like how you linked them to jump to each section - nicely done!

Seen here: https://github.com/nanoanno/teacozy/blob/master/resources/css/style.css#L65-L66

Since these lines are repeated often in your code - I recommend putting them into a class styling and applying the class to all the elements where they were used.

Great job on this project - I really liked how you styled the nav, as well as how you chose to organize your HTML (it's very clean and readable!). In terms of improvement, I recommend reducing repetitive code in your CSS, and migrating your opacity to the images from the body.

If you’re in the mood for a challenge, I would look into implementing a mobile version of this page! Here's a link to get you started: https://www.w3schools.com/css/css_rwd_mediaqueries.asp. Keep up the stellar work! ^_^

Seen here: https://github.com/nanoanno/teacozy/blob/master/resources/css/style.css#L6

Since the body will affect the opacity of the text, and the text is already difficult to read on a dark background, I recommend setting it to the images, to make it easier to read the overlaying text.