Skip to content

Commit

Permalink
Merge pull request w3c#213 from blazkovicz/custom-missing-text
Browse files Browse the repository at this point in the history
missing_text now an arg for the 'missing-data' chart
  • Loading branch information
hamilton committed Nov 20, 2014
2 parents 42fd2e7 + cb0c28e commit a8ac673
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 13 deletions.
9 changes: 4 additions & 5 deletions js/metricsgraphics.js
Expand Up @@ -131,7 +131,8 @@ function data_graphic() {
right: 0,
left: 0,
width: 350,
height: 220
height: 220,
missing_text: 'Data currently missing or unavailable'
}

var args = arguments[0];
Expand Down Expand Up @@ -2345,17 +2346,15 @@ charts.missing = function(args) {
.attr('width', args.width - (args.left * 2))
.attr('height', args.height - (args.top * 2));

var missing_text = 'Data currently missing or unavailable';

// add missing text
svg.selectAll('.missing_text').data([missing_text])
svg.selectAll('.missing_text').data([args.missing_text])
.enter().append('text')
.attr('class', 'missing-text')
.attr('x', args.width / 2)
.attr('y', args.height / 2)
.attr('dy', '.50em')
.attr('text-anchor', 'middle')
.text(missing_text)
.text(args.missing_text)

return this;
}
Expand Down
6 changes: 3 additions & 3 deletions js/metricsgraphics.min.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

6 changes: 2 additions & 4 deletions src/charts/missing.js
Expand Up @@ -26,17 +26,15 @@ charts.missing = function(args) {
.attr('width', args.width - (args.left * 2))
.attr('height', args.height - (args.top * 2));

var missing_text = 'Data currently missing or unavailable';

// add missing text
svg.selectAll('.missing_text').data([missing_text])
svg.selectAll('.missing_text').data([args.missing_text])
.enter().append('text')
.attr('class', 'missing-text')
.attr('x', args.width / 2)
.attr('y', args.height / 2)
.attr('dy', '.50em')
.attr('text-anchor', 'middle')
.text(missing_text)
.text(args.missing_text)

return this;
}
Expand Down
3 changes: 2 additions & 1 deletion src/common/data_graphic.js
Expand Up @@ -132,7 +132,8 @@ function data_graphic() {
right: 0,
left: 0,
width: 350,
height: 220
height: 220,
missing_text: 'Data currently missing or unavailable'
}

var args = arguments[0];
Expand Down

0 comments on commit a8ac673

Please sign in to comment.