@AlvaroPrates
Posted
I'm fairly new to web-development, but I think I can leave a little feedback.
When I've done this project, I forgot to add the box-sizing: border-box
CSS property and had some trouble to adjust the width. So, nice catch implementing it!
Regarding the topics that you need help:
- Instead of wrapping the image, title and text within normal
div
s, maybe you can use the<article>
and/or<figure>
HTML semantic tags. - I didn't use any media queries to this project. Do you really need any?
Marked as helpful
@py-code314
Posted
Thank you. I've changed the <div>
to <article>
as it makes more sense semantically.
I've noticed in the figma file that title text has 'Drop shadow' in Desktop version and the Mobile version doesn't. That's why I had to add @media
in there