@astridv
Posted
Hi Alan!
Good job on the challenge 👏 I think it looks very nice on both desktop and mobile 😄 The transition from large to smaller screens is a bit choppy (first it is one size before jumping to another). I actually think you could simplify your code by removing your media queries, and just keeping the max-width
property in this case as the card layout doesn't change.
The design looks really good. To get it even closer you could consider reducing the padding in your person-bio
class to remove some white space.
Also, quick note on the strong
tag. This tag is used to define text with strong importance in a document. If the text isn't of special importance, it is better to use the <b>
tag or add a css style with font-weight: bold
.
Hope this helps and keep up the good work 😄