Rio Cantre• 9,690
@RioCantre
Posted
Hello there! Good job in this challenge. Viewing the solution, I think you should consider the following as well…
- Import the
attribution
style in CSS file and remove thestyle
tag - Include description with the
alt
in image tags. Like this line<img class="hero-img" src="./images/illustration-hero.svg" alt="">
- Add the background image in the
body
tag.image/pattern-background-desktop.svg
withbackground-size: cover/contain;
,background-position: top;
andbackground-repeat: no-repeat;
- Add the hover state of the main button in
.payment
withcursor:pointer;
andbackground-color: hsla(245, 78%, 79%, 1);
- Add
cursor: pointer
for the links - Refactor this part and wrap it with
footer
tag
<section>
<div class="attribution">
Challenge by <a href="https://www.frontendmentor.io?ref=challenge" target="_blank">Frontend Mentor</a>.
Coded by <a href="https://github.com/rouftarek/">RoufTarek</a>.
</div>
</section>
Into:
<footer class="attribution">
Challenge by <a href="https://www.frontendmentor.io?ref=challenge" target="_blank">Frontend Mentor</a>.
Coded by <a href="https://github.com/rouftarek/">RoufTarek</a>.
</footer>
Above all, Well done! Keep it going!
0
Abdur Rouf Tarek• 120
@rouftarek
Posted
@RioCantre Thank you again. This is very helpful.
0