# Code Review: Learning Journal
### _Code by: Ajo ,_
### _Reviewer: Felicity Evans_

## What you did well
- Concern for code structure, naming conventions
- Responsive design (mobile first)
- Well commented CSS
- Use of grid-template-area

## Questions / Concerns
1) I'm trying to figure out what's "best practice" when it comes to setting margins, e.g. margins on single elements vs. on larger sections, on headings vs. paragraphs etc. Are there any standards/guidelines to follow? Am I on the right track?
2) I made three separate html files and one single css file. Maybe a bit confusing?

## What could be improved?
- Names of html files (index is default page)
- Include Nav element
- Class names need not be unique: blog-two, blog-three or unique to page e.g. index-1
    - Instead think about functionality / Component-based
- Convention to use grid for main areas of page but flexbox for subsections

    
## Challenge
- Alter CSS to use flexbox