@RioCantre
Posted
Hello there! Good job with this challenge. Viewing your solution, I think you should considered the following...
- the hover state of the main button into this...
.payment-btn:hover {
background: hsla(245, 78%, 79%, 1);
cursor: pointer;
}
- Add the background image in the
body
rule set like this...
body {
min-height: 100vh;
font-size: 16px;
font-family: "Red Hat Display", sans-serif;
background-color: var(--pale-blue);
display: grid;
place-items: center;
color: var(--desaturated-blue);
background-image: url(images/pattern-background-desktop.svg);
background-repeat: no-repeat;
}
and adjust the image in media queries for mobile
- Wrap the
attribution
withfooter
tag
Hope this helps and Keep up the good work!
Marked as helpful
@DeadParadox
Posted
@RioCantre Right, I completely forgot about background-image. I was stuck on how to implement the background efficiently, and yeah, I forgot to swap to a footer tag but I didn't feel like pushing a commit just for that. And if there was a hover state in the challenge, I must've missed it, good call!
Thanks for the feedback, much appreciated! :)