Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

So few additions, built by flexbox mainly. "Profile card component"

@RyanAbdaul

Desktop design screenshot for the Profile card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Guys I have a question for you!

Okay first of all, I feel that the way I put the width of the card is wrong, Idk I feel it is, so if it is wrong, tell me is there anyway better than mine?>

and if there is, don't tell me, just give me the keyword and I will search <; I'm waiting for your comments, peace be with you❤

Community feedback

Daniel 🛸 44,230

@danielmrz-dev

Posted

Hello @RyanAbdaul!

Your solution looks great!

The background pattern with the circles is a bit tricky, but you don't need separate containers to create it. It's possible to work with more than one image on the property background. Here's how you can do it:

📌 Apply this on the body:

  background-color: var(--Dark-cyan);
  background-image: url("./images/bg-pattern-top.svg"), url("./images/bg-pattern-bottom.svg");
  background-repeat: no-repeat, no-repeat;
  background-position: right 52vw bottom 35vh, left 48vw top 52vh;

I hope it helps!

Other than that, you did an excelent job!

Marked as helpful

0
Marzuk Sanni 1,360

@Zukizuk

Posted

Hello @RYAN

Congratulation on completing your project.

Your solution looks nice.

I have few suggestions that I think might interest you.

First of all, I don't think there's a problem with your width. But you could have simply done it width and not max-width and everything will work fine. Better still you can use viewport unit like vw vh and that will help you with responsiveness too.

Now, I have few suggestions that i think might helpful, consider using as a background image with the help of css property background or background-image together with background-size and background-position. Example use this

body {
  background: url(../images/bg-pattern-top.svg) no-repeat,
    url(../images/bg-pattern-top.svg) no-repeat, var(--Dark-cyan);
}

instead of using the images meant for the background in the html direclty since the images are purely presentational and not a content.

Also consider using semantic hmtl in your codes like <main></main><header></header> instead of div for the wrapper. So i would like you to change this <div class="container"></div> <main class="container"></main>

I hope this feedback was helpful

Marked as helpful

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join 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