jdiorio2393 / prj-rev-bwfs-dasmoto Goto Github PK

Great! I really love your styling and layout choices as they compliment the website idea very well. It goes beyond the specification which makes improvements on the original layout. On top of all of this, your HTML and CSS remained relatively small in size which is good to see.

Simplify - I recommend adding alt attributes to your images. The purpose of alt attributes is to provide a text description of the contents of an image file - it also helps with accessibility.

Read more here: https://www.w3.org/wiki/HTML/Usage/TextAlternatives

Here is a more in-depth explanation on alt & accessibility if you are curious: https://webaim.org/techniques/alttext/

Grade: Exceeds expectations

Overall, this project has exceeded expectations. The HTML and CSS remained relatively small in size for big improvements to the website specification. The layout choices compliment the website really well. One thing I would change about this project is making sure you add alt attributes to your images so we can provide our users with a text descriptor of the image. Next steps, I would look into learning about media queries in an effort to make your pages responsive so they will be viewable on all types of media such as phone, PC, laptop, etc: https://developer.mozilla.org/en-US/docs/Web/CSS/Media_Queries/Using_media_queries

Nicely done!

Great! The usage of flexbox shows your ability and attention to important aspects surrounding responsive websites. Additionally, you are using grouped selectors and even some psuedo-selectors like :hover which is awesome to see.