Peer to Peer Code Review

cmolinan / Portfolio-project

This Portfolio was built entirely in HTML/CSS and JavaScript for DOM Manipulation, using technologies such as Flexbox and Grid, and applying Responsive design. It is based on a FIGMA design

https://cmolinan.github.io/

4 stars 0 forks source link

Peer to Peer Code Review #12

Open RodChak opened 2 years ago

RodChak commented 2 years ago

Suggestion from Morning Team

Please consider centering the elements in your navbar so they align, I suggest you use the justify_content align property.

It Would be good if you consider these changes

limavfabio commented 2 years ago

More specifically this code block on line 20 <div class="menu-desktop"> <a href="#" class="menu-opt">Portfolio</a> <a href="#" class="menu-opt">About</a> <a href="#" class="menu.opt">Contact</a> </div>

Maybe because of the menu being inside another container, the items didn't align correctly with the logo