Change the stroke

Question

i cant see how task 1 isnt processing

js/app.js

// height and width of the svg
var height = 800, width = 500;

var padding = 50;

var viz = d3.select("#viz-wrapper")
    .append('svg')
    .attr('id', 'viz')
    .attr('height', height)
    .attr('width', width);

d3.csv('app/climate_data_truncated.csv', function(data) {

    dots = viz.selectAll('circle')
        .data(data)
        .enter()
        .append('circle');

    dots.attr('r', function(d) {return Math.abs(d.TMIN) / 10})
        .attr('cx', function(d) {return Math.max(0 + padding, Math.random() * width - padding)})
        .attr('cy', function(d) {return Math.max(0 + padding, Math.random() * height - padding )})
        .style('stroke', function(d) {
            year = d.DATE.substring(0,3)
            if (year === "1973") {
                return "blue"
            }
            else 
                return "#ff00f5"
        .style('fill', function(d) {
            year = d.DATE.substring(0,4)
            if (year === "1973") {
                return "blue"
            }
            else {
                return "#ff00f5"
            }
        });
});

index.html

<!DOCTYPE html>
<html>
  <head>
    <meta charset="UTF-8">
    <title>Requests</title>
  </head>
  <body>
    <div id="viz-wrapper">
    </div>
  </body>
  <script src="js/d3.min.js" charset="utf-8"></script>
  <script src="js/app.js" charset="utf-8"></script>
</html>

Answer 1 · 2017-10-21T12:11:22Z

October 21, 2017 12:11pm

The new code has a few issues:

it is missing the closing brace of the new function
it is missing the closing parenthesis of the "style" call
it has the wrong 2nd argument for the "substring" method

Tip: To confirm that parentheses or braces are balanced, count them to be sure you have the same number of opens and closes.

Welcome to the Treehouse Community

Looking to learn something new?

Scott Glover

Scott Glover

Change the stroke

1 Answer

Steven Parker

Steven Parker

Scott Glover

Scott Glover