Skip to content

Commit

Permalink
fixed y_axis failing test and set defaultYMin to true
Browse files Browse the repository at this point in the history
  • Loading branch information
stormpython committed Jan 19, 2015
1 parent e6856e5 commit efb5858
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/kibana/components/vislib/vis.js
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ define(function (require) {
this.el = $el.get ? $el.get(0) : $el;
this.ChartClass = chartTypes[config.type];
this._attr = _.defaults(config || {}, {
defaultYMin: false
defaultYMin: true
});
this.eventTypes = {
enabled: []
Expand Down
5 changes: 4 additions & 1 deletion test/unit/specs/vislib/lib/y_axis.js
Original file line number Diff line number Diff line change
Expand Up @@ -137,9 +137,12 @@ define(function (require) {
yAxisDiv = el.append('div')
.attr('class', 'y-axis-div');

dataObj = new Data(data, {});
dataObj = new Data(data, {
defaultYMin: true
});
yAxis = new YAxis({
el: node,
yMin: dataObj.getYMinValue(),
yMax: dataObj.getYMaxValue(),
_attr: {
margin: { top: 0, right: 0, bottom: 0, left: 0 }
Expand Down

0 comments on commit efb5858

Please sign in to comment.