search

neiljb65 / prj-rev-bwfs-dasmoto

0 stars 0 forks source link

SUMMARY #1

Open gdedi001 opened 7 years ago

gdedi001 commented 7 years ago

Grade: Satisfactory

Great job on your project - I loved how you were able to incorporate the <p> and <span> tag making your HTML very organized! In addition to this, you utilized the class attribute properly which allowed you to incorporate the blue font color. These two aspects of your code helped you meet the design requirements! An area for improvement here is with respect to how you utilized the class attribute in some situations. For the headers you should have opted for id instead.

A recommendation here is to review the following article. Try applying the concept and its usage. It will aid you as your projects get more complex. Read it thoroughly and view the examples! Keep up the great work! id vs. class

neiljb65 commented 7 years ago

Thanks for the valuable feedback Gabriel, I'll update my code accordingly.

Cheers, Neil

On Sat, Aug 19, 2017 at 10:00 AM, Gabriel de Diego <notifications@github.com

wrote:

Grade: Satisfactory

Great job on your project - I loved how you were able to incorporate the

and tag making your HTML very organized! In addition to this, you utilized the class attribute properly which allowed you to incorporate the blue font color. These two aspects of your code helped you meet the design requirements! An area for improvement here is with respect to how you utilized the class attribute in some situations. For the headers you should have opted for id instead. A recommendation here is to review the following article. Try applying the concept and its usage. It will aid you as your projects get more complex. Read it thoroughly and view the examples! Keep up the great work! id vs. class — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub , or mute the thread .