@matbac85
Posted
Good job !
I'm a beginner so I can't be much help, but here are my comments
- It's best practice to create a separate file for CSS styles.
- Instead of a div for the part with the profile image, name and profession, you can use <figure> and <figcaption> elements.
- Your "alt" attribute lacks information.
- You forgot the shadow on each card.
- In your .container, I think it's unnecessary:
grid-auto-columns: 1fr;
grid-template-areas:
'one'
'two'
'three'
'four'
'five';
gap: 2.5rem;```
In mobile view, the cards are one below the other. This is the basic behaviour of the elements. With the code above, you're just repeating what exists by default. You can simply add this information when the screen width is larger and you need to position your cards according to a specific template.
I hope it helps !