All comments

• 

  P

  Nohelynaid• 180

  @Nohelynaid

  Submitted over 1 year ago

  It is actually working pretty well but somehow the background image cannot be displayed or its not compatible. FEEDBACK!

  

  Order summary solution

  2

  30

  

  Danilo de Oliveira• 70

  @Darilu

  Posted over 1 year ago

  hello, I tried to fix the problem in the background. and following this path, I was able to include the background image.

  body {
     margin: auto;
     font-family: 'Red Hat Display', sans-serif;
     text-align: center;
     background-image: url("../images/bg-desktop.svg");
     background-repeat: no-repeat;
     background-size: contain;
  }
  

  the repeat and size properties were just to get it right, but feel free to change it.

  Hope this helps.

  Marked as helpful

  0

  This comment has received 1 replyShow
• 

  Teekay Jnr• 40

  @TeekayJnr

  Submitted over 1 year ago

  Made this for only desktop. I dont really know how to make it responsive. Feedbacks will be appreciated.

  

  NFT Preview Card Component Solution

  1

  60

  

  Danilo de Oliveira• 70

  @Darilu

  Posted over 1 year ago

  congrats on the job!!

  I think that in this case there is not much to develop in terms of responsiveness.

  and I saw that you already use units of Relative Lengths and percentage in the width of the images. you are already on the right path.

  Sorry for the English, I used google translator to help me.

  (I'm also a beginner) ty

  0
• 

  Shady Omar• 950

  @Shady-Omar

  Submitted almost 3 years ago

  

  Order summary component challenge

  1

  20

  

  Danilo de Oliveira• 70

  @Darilu

  Posted almost 3 years ago

  Hi, very good your challenge, I'm also a beginner. And in my case the background image was proposed like this:

  background-repeat: no-repeat; background-size: contain;

  and i set the background color not to be white in the background

  background-color: hsl(225, 100%, 94%);

  Marked as helpful

  1

  This comment has received 1 replyShow