Hey! This was a fun challenged, this was my first time to make an approach to mobile first and make responsive. I was afraid about responsive. Welp more practice to improve.
Astrid
@astridvAll comments
- @AlanSeyfirSubmitted almost 4 years ago@astridvPosted almost 4 years ago
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 withfont-weight: bold
.Hope this helps and keep up the good work π
1