@felipeog
Posted
This comment was deleted
@kvcarido
Posted
@felipeog, thanks for taking the time to look it over and the really insightful feedback π
- I took your advice and utilized the
background-image
and respective properties instead of imgsβ I initially took this route, but didn't realize how specific I could get withbackground-position
and doubted my intuition. This time around, could I ask your opinion on my updated approach under thehtml
element? - Negative
margin-top
did the trick! Amazing π€© - Understood! With such a simple component I figured I could get away with it, but will keep this in mind going forward especially for bigger challenges!
This has helped tremendously! Appreciate you, Felipe! β
@felipeog
Posted
Hey, @kvcarido!
I think that the background solution is fine and responsive.
But if you want to go the extra mile, the design has a different size for the background on the mobile version. You can use a @media
query to apply different background values for different screen sizes.
Oh, and I forgot to comment on the use of CSS variables before, it is nice!
Keep hacking!