@AlanSeyfir
Submitted
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.
Looking to hire developers?
@astridv
@AlanSeyfir
Submitted
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.
@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 π