Noel Hoppe• 350
@noelhoppe
Posted
Hi,
- Use
min-h-screen
on<main>
to avoid bugs when the contents overflow - Set
p-10
to your card container because the padding is equal on all four sides - "London, United Kingdom" seems to be a
<p>
tag - Improve the link structure (use
<ul>
and<li>
instead of<div>
):
<ul>
<li><a href=""></a></li>
...
5. Use flexbox in your card instead of using so much margin properties.
Happy coding!
</ul>
Marked as helpful
0
Maryam Nasir• 150
@maryam-nasir
Posted
@noelhoppe Thank you so much for the feedback. Will surely look into it to improve my solution.
0