@Amer-10
Submitted
@luizfbn
@Amer-10
Submitted
@luizfbn
Posted
Really nice solution! Few things you could improve:
Good job and keep studying! :)
Marked as helpful
@ghamsin
Submitted
What are you most proud of, and what would you do differently next time?
hi i use resposive and variable font + give smoth animation to resposive title and link + hover animation on avatar img
What challenges did you encounter, and how did you overcome them?
to took alot of time learn variable font and use animation this but this satisfyed ilike result
What specific areas of your project would you like help with?
analysis the project spets and Structuring + clean coding and some tip to faster coding becus im slow as hell tnxxx ^_^ have a nice day
@luizfbn
Posted
Nice solution! I found some things you can improve:
About HTML:
About CSS:
Good job and keep studying! :)
What are you most proud of, and what would you do differently next time?
I liked the use of flexbox for the layout from the container to the child elements, sectioning with BEM.
What challenges did you encounter, and how did you overcome them?
In the author element, I found that I could not reduce the size of the image with flex-shrink. I solved it by putting the image in a and it could be reduced without problems.
What specific areas of your project would you like help with?
I would like help with CSS structure. I use Flex and utilities, but I am not clear if I have omitted some good practice or if it can be optimized in some way.
@luizfbn
Posted
Good solution. I found some stuffs that you can improve.
About HTML:
About style:
Tips:
Good job and keep studying! :)
Marked as helpful
@khaleelibraheem
Submitted
@luizfbn
Posted
Nice solution. It is accessible, responsive and has semantic HTML. The only thing I think can be improved is the card border needs to be bigger. Good job! :)