search

jkrueg75 / prj-rev-bwfs-tea-cozy

0 stars 0 forks source link

Summary #6

Open thecodeduchess opened 6 years ago

thecodeduchess commented 6 years ago

Grade: Meets Expectations

Summary: This was a great implementation of the project. First, I test your project by downloading it and viewing it in my own browser. Next, I go step by step with your page against the requirements. It matches to my specs exactly. After nitpicking through it, I did not find anything that requires a re-do. What I check for is code quality and how closely you can match to the spec.

The code was easily readable and met the required styling, class and ID naming. If you want to build onto this project, you could consider adding a border on some of the elements, changing the font design, background color or use other various font colors. Don't be afraid to experiment. You can make it your own for an exceeding expectations grade.

I hope you are enjoying your learning experience. You are certainly ready to move on to the next step!

jkrueg75 commented 6 years ago

Wonderful! Thanks so much! I wasn't aware that we could customize with these projects. I've been trying to duplicate these projects exactly as they appear. Thanks for the review.

thecodeduchess commented 6 years ago

Well, the sample layout is helpful and helps to re-enforce some skills, but you have free reign with colors, fonts and other minor designs. :)

On Sun, Dec 17, 2017 at 2:39 PM, Jon notifications@github.com wrote:

Wonderful! Thanks so much! I wasn't aware that we could customize with these projects. I've been trying to duplicate these projects exactly as they appear. Thanks for the review.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/jkrueg75/prj-rev-bwfs-tea-cozy/issues/6#issuecomment-352283695, or mute the thread https://github.com/notifications/unsubscribe-auth/AIUaFplauKFhWQ3W2HfpQkx2VMU6Vfjbks5tBXv7gaJpZM4REyFc .

-- Tell me and I forget. Show me and I remember. Involve me and I understand.

jkrueg75 commented 6 years ago

Good to know. Can’t wait to experiment! Thanks again.

On Sun, Dec 17, 2017 at 12:45 PM cyr3ne notifications@github.com wrote:

Well, the sample layout is helpful and helps to re-enforce some skills, but you have free reign with colors, fonts and other minor designs. :)

On Sun, Dec 17, 2017 at 2:39 PM, Jon notifications@github.com wrote:

Wonderful! Thanks so much! I wasn't aware that we could customize with these projects. I've been trying to duplicate these projects exactly as they appear. Thanks for the review.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub < https://github.com/jkrueg75/prj-rev-bwfs-tea-cozy/issues/6#issuecomment-352283695 , or mute the thread < https://github.com/notifications/unsubscribe-auth/AIUaFplauKFhWQ3W2HfpQkx2VMU6Vfjbks5tBXv7gaJpZM4REyFc

.

-- Tell me and I forget. Show me and I remember. Involve me and I understand.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/jkrueg75/prj-rev-bwfs-tea-cozy/issues/6#issuecomment-352284143, or mute the thread https://github.com/notifications/unsubscribe-auth/AVybO0h6dN-IXcFotPBXSyjg3i5o-HwEks5tBX1WgaJpZM4REyFc .