tommygebru
commented
5 years ago Suggestions:
-
Nice touch on adding opacity to the navigation bar, generally we ask students to stick the guidelines for the first two projects :wink: the capstone project however is more open to personal flair and design choices
-
Flex-grow is similar to using a percentage value or ratio, there is a great demo available in this article here flex-grow | CSS-Tricks
-
Sometimes CSS specificity, does come into play and may require creating stronger css selectors to overwrite previous styles, check out this great tool and resource on css specificity https://flukeout.github.io/
-
Also i think in the columns you could have used the
breaktags to space out text between lines, but your work is just fine as is 😉
NOTE: Always conduct a peer-review with your peers in Slack, this will improve performance and ability to problem solve! 💯
Your project contains little to no errors Biagio, I went ahead and reviewed your work for best practices and general areas of improvement 😃