If you don't set width, bullet chart gets container's.. height is static but can be changed manually

12 years ago · 94d44b7cec
parent c5dad2c36a
commit 94d44b7cec
3 changed files with 11 additions and 9 deletions
--- a/nv.d3.js
+++ b/nv.d3.js
@ -977,8 +977,8 @@ nv.models.bulletChart = function() {
      ranges = function(d) { return d.ranges },
      markers = function(d) { return d.markers },
      measures = function(d) { return d.measures },
-      width = 380,
-      height = 55,
+      width = null,
+      height = 80,
      tickFormat = null,
      tooltips = true,
      tooltip = function(key, x, y, e, graph) { 
@ -1006,7 +1006,8 @@ nv.models.bulletChart = function() {
  // For each small multipleâ€¦
  function chart(g) {
    g.each(function(d, i) {
-      var availableWidth = width - margin.left - margin.right,
+      var availableWidth = (width  || parseInt(container.style('width')) || 960)
+                             - margin.left - margin.right,
          availableHeight = height - margin.top - margin.bottom,
          that = this;

--- a/nv.d3.min.js
+++ b/nv.d3.min.js
--- a/src/models/bulletChart.js
+++ b/src/models/bulletChart.js
@ -9,8 +9,8 @@ nv.models.bulletChart = function() {
      ranges = function(d) { return d.ranges },
      markers = function(d) { return d.markers },
      measures = function(d) { return d.measures },
-      width = 380,
-      height = 55,
+      width = null,
+      height = 80,
      tickFormat = null,
      tooltips = true,
      tooltip = function(key, x, y, e, graph) { 
@ -38,7 +38,8 @@ nv.models.bulletChart = function() {
  // For each small multipleâ€¦
  function chart(g) {
    g.each(function(d, i) {
-      var availableWidth = width - margin.left - margin.right,
+      var availableWidth = (width  || parseInt(container.style('width')) || 960)
+                             - margin.left - margin.right,
          availableHeight = height - margin.top - margin.bottom,
          that = this;