Welcome to the Treehouse Community

Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.

Start your free trial

CSS Sass Basics (retired) Advanced Sass Concepts Advanced Mixin Arguments

Jason Welsh
seal-mask
.a{fill-rule:evenodd;}techdegree seal-36
Jason Welsh
Treehouse Project Reviewer
Posted by Jason Welsh
.a{fill-rule:evenodd;}techdegree seal-36
Jason Welsh
Treehouse Project Reviewer

Stuck on last problem

Here is my code: 

@mixin rainbow($colors...) {
  @each $color in $colors {
    .#{$color} {
      background: $colors;
    }
  }
}

.div {
  @include rainbow(red, orange, yellow, green);
}

And this is the error it is returning:

'The <div> with the class .red should have a red background. Did you use red as a parameter when including the rainbow mixin?'

Not sure how to fix it

index.html 
<!DOCTYPE html>
<html>
<head>
  <title>Sass Basics - Code Challenge</title>
  <link rel="stylesheet" type="text/css" href="page-style.css">
  <link rel="stylesheet" type="text/css" href="style.css">
</head>
<body>
  <div class="box"></div>
  <div class="red"></div>
  <div class="orange"></div>
  <div class="yellow"></div>
  <div class="green"></div>
</body>
</html>
style.scss 
/* Write your SCSS code below. */

@mixin square($size, $color: black) {
  height: $size;
  width: $size;
  border: 1px solid $color;
}

.box {
  @include square($color: red, $size: 10px);
}

@mixin rainbow($colors...) {
  @each $color in $colors {
    .#{$color} {
      background: $colors;
    }
  }
}

.div {
  @include rainbow(red, orange, yellow, green);
}

1 Answer

laurahaywood
seal-mask
.a{fill-rule:evenodd;}techdegree
laurahaywood
Full Stack JavaScript Techdegree Student 9,937 Points
laurahaywood
.a{fill-rule:evenodd;}techdegree
laurahaywood
Full Stack JavaScript Techdegree Student 9,937 Points

When your .scss is refactored into .css the divs (red, orange, yellow, and green) do not have any dimensions so they will not show in the browser or preview. This can be solved by adding @include square; in your rainbow mixin. Quick side note if you don't want to include this you can still pass the challenge without it. Next if you remove the div selector around @include the boxes will display properly. So your scss should look like this

@mixin square ($size, $color: black ) {
  width: $size;
  height: $size;
  border: 1px solid $color;
}


@mixin rainbow ($colors...) {
  @each $color in $colors {
    .#{$color} {
        background-color: $color;
        @include square(10px, $color);
    }
  }
} 

.box {
  @include square($color: red, $size: 10px);

 }

@include rainbow (red, orange, yellow, green);