Rio Cantre• 9,690
@RioCantre
Posted
Hello there! Awesome work with this one. Viewing at your solution, I would recommend the following for you...
- Add
padding: 1rem;
in themain
rule set - Adjust the properties in
h1
rule set intofont-size: 1.2rem;
andline-height: 1.5rem;
- Adjust the font size of
p
intofont-size: 0.8rem;
- Include
alt
with description in theimg
tag - Refactor this part of the code:
<button>
<a href="https://facebook.com" target="_blank"
><img src="images\icon-facebook.svg"
/></a>
</button>
- Remove the `button` tag, it's improper to wrap the `a` with a `button`, it's good to use it as an alternative.
- Be careful with the use of slashes, use the correct way which is `/`
- Wrap the
card-lower
withdiv
instead ofsection
which need at least one heading hierarchy - Include
transition: all 200ms ease
in the button
Overall you did well, your CSS is fine. Keep practicing and Hope this helps! Cheers!
0