Skip to content
OverviewSolutions
5
Comments
4

Mario

@ohmymario130 points

Self-taught Web Developer

Latest solutions

Latest comments

  • Mario•130
    @ohmymario

    Looking good so far! Try adding hover states to your image, h1 and Jules Wyvern text. It also looks like your h1 has the wrong color. Try hsl(0, 0%, 100%) to match what is in the style guide.

    If you need to see an example on how to add hover states here is my solution

    Good luck! 😀

    Marked as helpful
  • Mario•130
    @ohmymario

    Great job on this!

    I would suggest maybe adding a background-color to your body element. Your background-image is partially transparent so it should show through. This should help with the bottom of your card getting lost in the background on desktop view. 😁

    Marked as helpful
  • Mario•130
    @ohmymario

    wow this looks incredible! I really enjoy the shrinking shadows when hovering over the headphones, speakers and earphones image links

    Congrats on getting that job 🎉

  • Mario•130
    @ohmymario

    Lookin good !

    Right now you have <img src="/images/image-qr-code.png" alt="" class="section--img--qr">

    try adding a period to your src <img src="./images/image-qr-code.png" alt="" class="section--img--qr">

    also don't forget to add something to your alt tag

    Marked as helpful