@Bayoumi-dev
Posted
Hey! It looks good!...
My suggestions:
An easy way to handle the background circles
, Give the body or the profile card container the following style to put these circles
as a background image
:
body {
//...
background: url("./images/bg-pattern-top.svg") right 50vw bottom 40vh no-repeat,
url("./images/bg-pattern-bottom.svg") 50vw 50vh no-repeat var(--dark-cyan); /* <---- Add */
}
.main-container::before { /* <---- Remove */
//...
}
.main-container::after { /* <---- Remove */
//...
}
- I suggest you put the status of the profile card into the
list item
to add moresemantics
to your project,Div
's don't do much for semantics but a list is much more meaningful..:
<ul class="stats">
<li><span class="stats-num">80K</span>Followers</li>
<li><span class="stats-num">803K</span>Likes</li>
<li><span class="stats-num"> 1.4K</span>Photos</li>
</ul>
Hope this help!... Keep it up👍
Marked as helpful
@Promise30
Posted
@Bayoumi-dev Thank you very much. Your comment has been really helpful, I will make the necessary changes and also implement them in my future projects.