@@ -75,6 +75,26 @@ angular.module('picardy.graphs.bar', ['picardy.graphs.common'])
75
75
} ) ;
76
76
}
77
77
78
+ function getYRange ( ) {
79
+ var range ;
80
+
81
+ if ( options . max && options . max . y ) {
82
+ range = [ 0 , options . max . y ] ;
83
+ } else {
84
+ range = [ 0 , d3 . max ( options . data , function ( d ) {
85
+ return d . y ;
86
+ } ) ] ;
87
+ }
88
+
89
+ return range ;
90
+ }
91
+
92
+ function getXRange ( ) {
93
+ return options . data . map ( function ( d ) {
94
+ return d . x ;
95
+ } ) ;
96
+ }
97
+
78
98
function drawYAxis ( done ) {
79
99
var axisLength = options . height ,
80
100
yAxis , range ;
@@ -83,27 +103,12 @@ angular.module('picardy.graphs.bar', ['picardy.graphs.common'])
83
103
axisLength -= margin . info ;
84
104
axisLength -= margin . top ;
85
105
y = d3 . scale . linear ( ) . range ( [ axisLength , 0 ] ) ;
86
-
87
- if ( options . max && options . max . y ) {
88
- range = [ 0 , options . max . y ] ;
89
- } else {
90
- range = [ 0 , d3 . max ( options . data , function ( d ) {
91
- return d . y ;
92
- } ) ] ;
93
- }
94
-
106
+ range = getYRange ( ) ;
95
107
y . domain ( range ) ;
96
108
} else {
97
109
axisLength -= margin . info + margin . label + 21 ;
98
110
y = d3 . scale . ordinal ( ) . rangeRoundBands ( [ 0 , axisLength ] , 0.2 ) ;
99
-
100
- if ( options . max && options . max . y ) {
101
- range = [ 0 , options . max . y ] ;
102
- } else {
103
- range = options . data . map ( function ( d ) {
104
- return d . x ;
105
- } ) ;
106
- }
111
+ range = getXRange ( ) ;
107
112
y . domain ( range ) ;
108
113
}
109
114
@@ -147,24 +152,11 @@ angular.module('picardy.graphs.bar', ['picardy.graphs.common'])
147
152
if ( isVertical ( ) ) {
148
153
axisLength -= margin . label + margin . yAxis ;
149
154
x = d3 . scale . ordinal ( ) . rangeRoundBands ( [ 0 , axisLength ] , 0.2 ) ;
150
- if ( options . max && options . max . x ) {
151
- range = [ 0 , options . max . x ] ;
152
- } else {
153
- range = options . data . map ( function ( d ) {
154
- return d . x ;
155
- } ) ;
156
- }
155
+ range = getXRange ( ) ;
157
156
x . domain ( range ) ;
158
157
} else {
159
158
x = d3 . scale . linear ( ) . range ( [ axisLength , 0 ] ) ;
160
- if ( options . max && options . max . y ) {
161
- range = [ 0 , options . max . y ] ;
162
- } else {
163
- range = [ 0 , d3 . max ( options . data , function ( d ) {
164
- return d . y ;
165
- } ) ] ;
166
- }
167
-
159
+ range = getYRange ( ) ;
168
160
x . domain ( range ) ;
169
161
}
170
162
0 commit comments