Skip to content
OverviewSolutions
3
Comments
3
Danilo de Oliveira
@Darilu

All comments

  • Nohelynaid•180
    @NohelynaidSubmitted 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
    Danilo de Oliveira•70
    @DariluPosted 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
  • Danilo de Oliveira•70
    @DariluPosted 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
  • Danilo de Oliveira•70
    @DariluPosted 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