search

PizzaGymDonut / ColmarAcademy

0 stars 0 forks source link

Responsive #4

Closed tommygebru closed 5 years ago

tommygebru commented 5 years ago

Really nice work on using multiple images for separate layouts Matthew, also really great use of fluid typography and responsive web design made possible with REM Implementing REM in CSS (or, How to Be a Shiny Happy Developer) - Words and Code

Your transition between different layouts can be smoother, great use of the entire pages length and width. 👍

tommygebru commented 5 years ago

Suggestions:

Remove the body width, within the mobile media query, this will prevent your content from being cut and hidden 👀

Also you can remove the bullet points from your unordered list (courses section in the mobile layout) on line 186 of your CSS For example

list-style-type: none;

And from there you can re-center your list-item content with 

.list ul{
  align-items: flex-start;
  justify-content: flex-start;
}