Overall Summary

[jlag34]

Criteria 1: HTML Structure

• _Score Level: 4/4
• _Comment(s): Your HTML was done really well. You had good indentation, use of wrapping elements, comments and variety of elements.

Criteria 2: Visual Layout for Both Desktop and Mobile Sizes

• _Score Level: 4/4
• _Comment(s): Both of your layouts looked really nice. Nothing broke their containers and the sections formatted nicely.

Criteria 3: Responsive Design (Media Queries, Responsive Units, etc.)

• _Score Level: 3/4
• _Comment(s): For the most part, the elements moving from desktop to mobile collapsed nicely. There were a few areas, like the nav bar, that should have collapsed sooner for a clean collapse.

Criteria 4: Visual Design and Accessibility (Color Palette, Typography, Transitions, etc.)

• _Score Level: 1/4
• _Comment(s): I didn't notice any hover effects or animations on your page. Everything but a few hyperlinks were black, white and gray. Would have been nice to see a little color added in.

Overall Score: 12/16

Overall, you did a good job. The focus of this project was on media queries and you handled them nicely. Your layout for both views was very good and just needed a little extra polish on how they collapsed. Good job on your HTML, everything was really nice. The only thing your project lacked was a bit of styling to bring it to life. Adding a nice color scheme and a few hover effects would really make it pop. Other than that, great job.

[naicox]

Thanks for your review. Here are some of my considerations: For criteria 3: I feel I could be more efficient in the code, like if I'd used too many selectors. The same feeling in the css code.
For criteria 4: I just didn't realise I had to make it fancy, neither had the time xD.

[naicox]

So do you have any particular tip for these issues or I should just don't worry about it?