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

Community feedback

• 

  Daniel 🛸•44,270

  @danielmrz-devPosted 10 months ago

  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

  @ZukizukPosted 10 months ago

  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