Commits

Anonymous committed 166be8b

Experimenting with way to have positive bars start at 0.

Comments (0)

Files changed (1)

src/jqplot.core.js

                 }
                 $.extend(true, temp, {seriesColors:this.seriesColors, negativeSeriesColors:this.negativeSeriesColors}, this.options.seriesDefaults, this.options.series[i]);
                 var dir = 'vertical';
-                if (temp.renderer.constructor == $.jqplot.barRenderer && temp.rendererOptions && temp.rendererOptions.barDirection == 'horizontal') {
+                if (temp.renderer === $.jqplot.BarRenderer && temp.rendererOptions && temp.rendererOptions.barDirection == 'horizontal') {
                     dir = 'horizontal';
                 }
                 temp.data = normalizeData(this.data[i], dir, this.defaultAxisStart);
                     temp._yaxis.show = true;
                 }
 
+                // Hack to not pad out bottom of bar plots unless user has specified a padding.
+                // if (temp.renderer === $.jqplot.BarRenderer) {
+                //     var pm = false;
+                //     switch (dir) {
+                //         case 'vertical':
+                //             if (this.options.axesDefaults && (this.options.axesDefaults.pad != null || this.options.axesDefaults.padMin != null)) {
+                //                 pm = true;
+                //             }
+                //             else if (this.options.axes && this.options.axes[this.yaxis] && (this.options.axes[this.yaxis].pad != null || this.options.axes[this.yaxis].padMin != null)) {
+                //                 pm = true;
+                //             }
+                //             if (!pm) {
+                //                 temp._yaxis.padMin = 1.0;
+                //                 temp._yaxis.rendererOptions.forceTickAt0 = true;
+                //             }
+                //             break;
+
+                //         case 'horizontal':
+                //             if (this.options.axesDefaults && (this.options.axesDefaults.pad != null || this.options.axesDefaults.padMin != null)) {
+                //                 pm = true;
+                //             }
+                //             else if (this.options.axes && this.options.axes[this.xaxis] && (this.options.axes[this.xaxis].pad != null || this.options.axes[this.xaxis].padMin != null)) {
+                //                 pm = true;
+                //             }
+                //             if (!pm) {
+                //                 temp._xaxis.padMin = 1.0;
+                //                 temp._xaxis.rendererOptions.forceTickAt0 = true;
+                //             }
+                //             break;
+                //     }
+
+                // }
+
                 // parse the renderer options and apply default colors if not provided
                 if (!temp.color && temp.show != false) {
                     temp.color = cg.next();