Grammar police: fixing typos

master-patched
Lee Gillentine 11 years ago
parent 2ac200ead0
commit a89a5ce29a

@ -40,7 +40,7 @@ var chart = nv.models.bullet()
*/
];
//TODO: to be consistent with other models, shouild be appending a g to an already made svg, not creating the svg element
//TODO: to be consistent with other models, should be appending a g to an already made svg, not creating the svg element
var vis = d3.select("#chart").selectAll("svg")
.data(data)
.enter().append("svg")

@ -41,7 +41,7 @@ var chart = nv.models.bulletChart()
*/
];
//TODO: to be consistent with other models, shouild be appending a g to an already made svg, not creating the svg element
//TODO: to be consistent with other models, should be appending a g to an already made svg, not creating the svg element
var vis = d3.select("#chart").selectAll("svg")
.data(data)
.enter().append("svg")

@ -48,7 +48,7 @@ svg {
<script>
// Wrapping in nv.addGraph allows for '0 timeout render', stors rendered charts in nv.graphs, and may do more in the future... it's NOT required
// Wrapping in nv.addGraph allows for '0 timeout render', stores rendered charts in nv.graphs, and may do more in the future... it's NOT required
var chart;
nv.addGraph(function() {

@ -67,7 +67,7 @@ svg > g > path {
//TODO: create a nv.models.horizonChart model
// (if need be an nv.models.horizon model as well, but might get away with jsut using d3.horizon)
// (if need be an nv.models.horizon model as well, but might get away with just using d3.horizon)
// the horizontChart model should have a "tooltip" like functionality that shows a point over the
// current value based on the mouse's x position

@ -53,7 +53,7 @@ svg {
nv.addGraph(function() {
var chart = nv.models.lineChart();
chart.xAxis // chart sub-models (ie. xAxis, yAxis, etc) when accessed directly, return themselves, not the partent chart, so need to chain separately
chart.xAxis // chart sub-models (ie. xAxis, yAxis, etc) when accessed directly, return themselves, not the parent chart, so need to chain separately
.tickFormat(d3.format(',r'));
chart.yAxis

@ -46,7 +46,7 @@ nv.log = function() {
nv.render = function render(step) {
step = step || 1; // number of graphs to generate in each timout loop
step = step || 1; // number of graphs to generate in each timeout loop
render.active = true;
nv.dispatch.render_start();

@ -586,7 +586,7 @@ nv.models.cumulativeLineChart = function() {
//TODO: implement check below, and disable series if series loses 100% or more cause divide by 0 issue
if (v < -.95) {
//if a series loses more than 100%, calculations fail.. anything close can cause major distortion (but is mathematically currect till it hits 100)
//if a series loses more than 100%, calculations fail.. anything close can cause major distortion (but is mathematically correct till it hits 100)
line.tempDisabled = true;
return line;
}

@ -14,7 +14,7 @@ nv.models.line = function() {
, color = nv.utils.defaultColor() // a function that returns a color
, getX = function(d) { return d.x } // accessor to get the x value from a data point
, getY = function(d) { return d.y } // accessor to get the y value from a data point
, defined = function(d,i) { return !isNaN(getY(d,i)) && getY(d,i) !== null } // allows a line to be not continous when it is not defined
, defined = function(d,i) { return !isNaN(getY(d,i)) && getY(d,i) !== null } // allows a line to be not continuous when it is not defined
, isArea = function(d) { return d.area } // decides if a line is an area or just a line
, clipEdge = false // if true, masks lines within x and y scale
, x //can be accessed via chart.xScale()

@ -15,7 +15,7 @@ nv.models.line = function() {
var scatter = nv.models.scatter()
.id(id)
.size(16) // default size
.sizeDomain([16,256]), //set to speed up calculation, needs to be unset if there is a cstom size accessor
.sizeDomain([16,256]), //set to speed up calculation, needs to be unset if there is a custom size accessor
//x = scatter.xScale(),
//y = scatter.yScale(),
x, y,

@ -9,7 +9,7 @@ nv.models.scatter = function() {
, width = 960
, height = 500
, color = nv.utils.defaultColor() // chooses color
, id = Math.floor(Math.random() * 100000) //Create semi-unique ID incase user doesn't selet one
, id = Math.floor(Math.random() * 100000) //Create semi-unique ID incase user doesn't select one
, x = d3.scale.linear()
, y = d3.scale.linear()
, z = d3.scale.linear() //linear because d3.svg.shape.size is treated as area
@ -21,7 +21,7 @@ nv.models.scatter = function() {
, forceX = [] // List of numbers to Force into the X scale (ie. 0, or a max / min, etc.)
, forceY = [] // List of numbers to Force into the Y scale
, forceSize = [] // List of numbers to Force into the Size scale
, interactive = true // If true, plots a voronoi overlay for advanced point interection
, interactive = true // If true, plots a voronoi overlay for advanced point intersection
, pointActive = function(d) { return !d.notActive } // any points that return false will be filtered out
, padData = false // If true, adds half a data points width to front and back, for lining up a line chart with a bar chart
, clipEdge = false // if true, masks points within x and y scale

@ -28,7 +28,7 @@ nv.utils.windowSize = function() {
// Easy way to bind multiple functions to window.onresize
// TODO: give a way to remove a function after its bound, other than removing alkl of them
// TODO: give a way to remove a function after its bound, other than removing all of them
nv.utils.windowResize = function(fun){
var oldresize = window.onresize;

Loading…
Cancel
Save