Learn CSS Variables by Building a City Skyline - Step 41

Tell us what’s happening:

Whats wrong with this code, aren’t all the values set in place correct?

Your code so far

<!-- file: index.html -->
<!DOCTYPE html>
<html lang="en">    
  <head>
    <meta charset="UTF-8">
    <title>City Skyline</title>
    <link href="styles.css" rel="stylesheet" />   
  </head>

  <body>
    <div class="background-buildings">
      <div></div>
      <div></div>
      <div class="bb1">
        <div class="bb1a"></div>
        <div class="bb1b"></div>
        <div class="bb1c"></div>
        <div class="bb1d"></div>
      </div>
      <div class="bb2"></div>
      <div class="bb3"></div>
      <div></div>
      <div class="bb4"></div>
      <div></div>
      <div></div>
    </div>

    <div class="foreground-buildings">
      <div></div>
      <div></div>
      <div class="fb1"></div>
      <div class="fb2"></div>
      <div></div>
      <div class="fb3"></div>
      <div class="fb4"></div>
      <div class="fb5"></div>
      <div class="fb6"></div>
      <div></div>
      <div></div>
    </div>
  </body>
</html>
/* file: styles.css */
:root {
  --building-color1: #aa80ff;
  --building-color2: #66cc99;
  --building-color3: #cc6699;
  --building-color4: #538cc6;
  --window-color1: black;
}

* {
  border: 1px solid black;
  box-sizing: border-box;
}

body {
  height: 100vh;
  margin: 0;
  overflow: hidden;
}

.background-buildings, .foreground-buildings {
  width: 100%;
  height: 100%;
  display: flex;
  align-items: flex-end;
  justify-content: space-evenly;
  position: absolute;
  top: 0;
}

/* BACKGROUND BUILDINGS - "bb" stands for "background building" */
.bb1 {
  width: 10%;
  height: 70%;
  display: flex;
  flex-direction: column;
  align-items: center;
}

/* User Editable Region */

.bb1-window, .bb1a {
  width: 70%;
  height: 10%;
  background-color: var(--building-color1);
  background: linear-gradient(
      var(--building-color1),
      var(--window-color1)
    );
}


.bb1b {
  width: 80%;
  height: 10%;
  background-color: var(--building-color1);
}

.bb1c {
  width: 90%;
  height: 10%;
  background-color: var(--building-color1);
}

.bb1d {
  width: 100%;
  height: 70%;
  background-color: var(--building-color1);
}



/* User Editable Region */

.bb2 {
  width: 10%;
  height: 50%;
  background-color: var(--building-color2);
}

.bb3 {
  width: 10%;
  height: 55%;
  background-color: var(--building-color3);
}

.bb4 {
  width: 11%;
  height: 58%;
  background-color: var(--building-color4);
}

/* FOREGROUND BUILDINGS - "fb" stands for "foreground building" */
.fb1 {
  width: 10%;
  height: 60%;
  background-color: var(--building-color4);
}

.fb2 {
  width: 10%;
  height: 40%;
  background-color: var(--building-color3);
}

.fb3 {
  width: 10%;
  height: 35%;
  background-color: var(--building-color1);
}

.fb4 {
  width: 8%;
  height: 45%;
  background-color: var(--building-color1);
  position: relative;
  left: 10%;
}

.fb5 {
  width: 10%;
  height: 33%;
  background-color: var(--building-color2);
  position: relative;
  right: 10%;
}

.fb6 {
  width: 9%;
  height: 38%;
  background-color: var(--building-color3);
}
    

Your browser information:

User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/129.0.0.0 Safari/537.36

Challenge Information:

Learn CSS Variables by Building a City Skyline - Step 41

Hi @booleanmethod9

You should create a new class selector called bb1-window .

Try removing the second selector.

Happy coding

Thanks, but it doesn’t seem to work, the prompt asks for the property of bb1a to be moved to bb1-window

So you’ll need to move that code.

moved it but it did not work

What’s the hint message?

You should move the height property and value from .bb1a to .bb1-window .

Did you copy and paste, cut and paste, or something else?

I put the properties of bb1a into bb1-window and deleted bb1a

Did you delete just the property asked in the instructions, or the whole selector?

I could imagine, the prompt is going to ask i remove the other properties next and put it in bb1-window… I added it back and deleted the height, then it moved on to asking I remove the background

The new selector should only have two properties.
The other selector should have two fewer properties.

Please reset the step to restore the original code and try again.

1 Like

But it seems to ask to move the properties of bb1a to bb1-window:
You want to add the same gradient to the next two sections. Instead of doing that, create a new class selector called bb1-window ,----- and move the height and background properties and values from .bb1a to the new class selector.

Wow, what you said works. But why would the prompt ask for ‘and values’…

1 Like

Example

selector {
property: value;
.....
};

more than one property/value pairs is called properties/values

Oh yes, thank you for that

2 Likes

And thank you Teller for sticking with me

2 Likes

you are required to

  1. create a new class bb1-window
  2. move the height background property and it`s values to the new class (cut completely ) from .bb1a