@FluffyKas
Posted
Heyo,
That looks great! I haven't completed this challenge yet, so not sure if it includes some hover styles for the button, but regardless it would be nice to see some, even if you had to come up with it yourself ^^
You also have a somewhat unnecessary alt text on your cart icon (its just a decorative image), you could leave the alt text empty so screen readers don't read it out needlessly.
The border-radius is also a tiny bit messed up in mobile view (works fine in desktop view, so I guess just needs a media query).
So overall really well done, it's a neat solution!
@arrejoria
Posted
@FluffyKas thanks for the advice. It was really late, when i commited the solution i dont realice that my layout in mobile was that bad haha
Very happy for your comment! β€οΈ
PS: in the layot the button has an active style, only change bg color