Tea Cozy Project | Front End Engineer career path project

Projects Skill Path Projects
1

I just finished the Tea Cozy project. My code is here: GitHub - AmyWilford/TeaShop

Happy to swap code-reviewing duties for anyone else who has recently finished the project.

Open to any and all feedback

1 Like
2

Hi, I would love to give you some feedback, but do you have any live version of the website ?
If not maybe you can try to upload it to your Github pages.

3

Hi Chris,

Thanks for your reply! Here is a live version of the site: https://amywilford.github.io/TeaShop/

4

Hey @arc0192428506,

I just had a chance to take a look at your project. It looks pretty great! I think you’re definitely on the right track!

It’s been a little while since I completed this project but I’d love to get a code review if you don’t mind? I did revisit this project recently though for a bit of restyling.
You can find my repo @: GitHub - DakotahGodfrey/codecademy_tea_cozy: Codecademy intermediate CSS project.
Live Demo: https://objective-pasteur-ebfe86.netlify.app/

The only thing I might suggest is trying to increase the contrast for text. It looks like you’re using an overlay on the universal selector:

* {
    font-family: Helvetica, sans-serif;
    color: seashell;
    background-color: black;
    text-align: center;
    opacity:0.9;

}

I think that might be what’s making the text a little harder to read.

5

Hiya, Thanks for taking a look at my code and site.

I love what you did with your site - and how you changed the styles for a sleeker look.

My only comments are, in design - if the header menu didn’t have an opacity it would make it clearer to read as the site scrolls. As for the code, I wonder if you could combine some of your classes to cut down on the amount of code you have to write overall. It also looks like you have some ID’s that were put in the html, but weren’t used in the css file - so you could remove those from the code.

Thanks again!

1 Like