@ayobanjo
Posted
Hello @villAsh. Congratulations on your first submission.
I do have a few corrections you could apply
HTML
-
For SEO, and proper semantics, you should use the
<picture>
tag. This will help you to also change from either mobile img to desktop img depending on what your approach is (mobile-first or not) picture tag -
The Gabrielle Essence Eau De Parfum should be an
h1
element, and the perfume a paragraph -
For the former price, you can simply use the
del
tag
CSS
-
The height of the
img
should be 100%, so it simply fills the available space in its container -
The perfume has a letter spacing, which you have omitted, try
letter-spacing: 0.25rem
, or whichever value you find desirable -
You can set a
max-width
for your main container on the desktop view.
Here is my own solution. I hope you find this helpful my submission
Marked as helpful
@villAsh
Posted
hello sir @ayobanjo, thank you for your kind advice.