florentinaspi / bwui-capstone-placeholder

1 stars 0 forks source link

Summary #5

Open jim-rosales opened 6 years ago

jim-rosales commented 6 years ago

Grade: Meets Expectations

Summary: Florentina, congrats on finishing your capstone project! Visually I loved the color branding you implemented for the site, and you did an excellent job making sure it was consistent throughout. You also made great use of CSS grid to layout your project.

A couple of things I did notice were broken images scattered around your site. Be careful with images as not having them rendered reduces the professionalism of a site along with any credibility it has. As well remember to make use of the alt attribute along with roles and aria-labels which help increase the accessibility of your site. For this consider revisiting the unit of accessibility to get a refresher on the importance of this topic.

To expand your project consider researching variables and analyzing how you can make use of them in your project. Variables are fantastic because they help increase the readability of your stylesheet while making it more adaptable to any future changes.

florentinaspi commented 6 years ago

Thank you! I will take your suggestions into account, and will continue my study to improve my knowledge.