Design comparison
Solution retrospective
any feedback about my best practice
Community feedback
- @RenszCamachoPosted almost 4 years ago
Hiya ππ» moibrahem98.
Good job mate πππ. You have done a fantastic job on this challenge π, place properly the background image is quite tricky.
Just a few suggestions IMHO.
-
Add an alternative text to your images.
<img class="person" src="images/image-victor.jpg" alt="victor" />
-
I would add a CSS reset, to avoid some annoying margins.
* { margin: 0; padding: 0; box-sizing: border-box; }
- To place the background-images:
body { background-image: url(../images/bg-pattern-top.svg), url(../images/bg-pattern-bottom.svg); background-position: right 50vw bottom 50vh, left 50vw top 50vh; background-repeat: no-repeat; background-color: hsl(185, 75%, 39%); display: flex; justify-content: center; align-items: center; width: 100%; height: 100vh; }
- To center the card.
.card__body { width: inherit; height: inherit; display: flex; justify-content: center; align-items: center; }
Happy codingπ§βπ»
1@moibrahem98Posted almost 4 years ago@RenszCamacho Words cannot express how grateful I am for your kindness and generosity <3 <3 <3
0 -
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord