Skip to content

Commit 2c3adca

Browse files
committed
Fixed lint issues
1 parent 7a75dd1 commit 2c3adca

File tree

2 files changed

+64
-66
lines changed

2 files changed

+64
-66
lines changed

app/directives/distribution-graph/distribution-graph.directive.js

Lines changed: 32 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
import angular from 'angular'
22
import d3 from 'd3'
3-
import React from 'react'
43
import ReactDOM from 'react-dom'
54
import Tooltip from 'appirio-tech-react-components/components/Tooltip/Tooltip.jsx'
65

@@ -179,19 +178,19 @@ import Tooltip from 'appirio-tech-react-components/components/Tooltip/Tooltip.js
179178
.attr('fill', function(d) {
180179
return ratingToColor($scope.colors, d.start)
181180
})
182-
ReactDOM.unmountComponentAtNode(document.getElementById('chart-tooltip'));
183-
ReactDOM.render(<Tooltip>
184-
<div className='tooltip-target'></div>
185-
<div className='tooltip-body'>
186-
<div className='tooltip-rating'></div>
187-
<div className='tooltip-challenge'>
188-
<div className='challenge-name'></div>
189-
<div className='challenge-date'></div>
190-
</div>
191-
</div>
192-
</Tooltip>
193-
, document.getElementById('chart-tooltip'));
194-
181+
ReactDOM.unmountComponentAtNode(document.getElementById('chart-tooltip'));
182+
ReactDOM.render(<Tooltip>
183+
<div className='tooltip-target'></div>
184+
<div className='tooltip-body'>
185+
<div className='tooltip-rating'></div>
186+
<div className='tooltip-challenge'>
187+
<div className='challenge-name'></div>
188+
<div className='challenge-date'></div>
189+
</div>
190+
</div>
191+
</Tooltip>
192+
, document.getElementById('chart-tooltip'));
193+
195194
svg.selectAll('rect.hover')
196195
.data(ranges)
197196
.enter()
@@ -213,32 +212,32 @@ import Tooltip from 'appirio-tech-react-components/components/Tooltip/Tooltip.js
213212
$scope.displayCoders = true
214213
$scope.numCoders = d.number
215214
$scope.$digest()
216-
217-
d3.select('#chart-tooltip')
218-
.style('left', (d3.event.pageX-2) + 'px')
215+
216+
d3.select('#chart-tooltip')
217+
.style('left', (d3.event.pageX-2) + 'px')
219218
.style('top', (d3.event.pageY-2) + 'px')
220-
.style('display', 'block')
221-
$('#chart-tooltip').addClass('distribution')
222-
d3.select('#chart-tooltip .tooltip-container')
223-
.style('left', '20px !important')
219+
.style('display', 'block')
220+
$('#chart-tooltip').addClass('distribution')
221+
d3.select('#chart-tooltip .tooltip-container')
222+
.style('left', '20px !important')
224223
.style('top', '-20px !important')
225-
.style('opacity', '1')
226-
d3.select('#chart-tooltip .tooltip-container .tooltip-pointer')
227-
.style('left', '-5.5px !important')
224+
.style('opacity', '1')
225+
d3.select('#chart-tooltip .tooltip-container .tooltip-pointer')
226+
.style('left', '-5.5px !important')
228227
.style('bottom', '25px !important')
229-
230-
d3.select('#chart-tooltip .challenge-name').text($scope.numCoders + ' Coders');
231-
d3.select('#chart-tooltip .challenge-date').text('Rating Range: '+ $scope.highlightedRating + '-'+($scope.highlightedRating+99));
232-
d3.select('#chart-tooltip .tooltip-rating').text($scope.numCoders);
233-
d3.select('#chart-tooltip .tooltip-rating').style('background', ratingToColor($scope.colors, $scope.highlightedRating));
234-
$('#chart-tooltip').show();
235-
228+
229+
d3.select('#chart-tooltip .challenge-name').text($scope.numCoders + ' Coders');
230+
d3.select('#chart-tooltip .challenge-date').text('Rating Range: '+ $scope.highlightedRating + '-'+($scope.highlightedRating+99));
231+
d3.select('#chart-tooltip .tooltip-rating').text($scope.numCoders);
232+
d3.select('#chart-tooltip .tooltip-rating').style('background', ratingToColor($scope.colors, $scope.highlightedRating));
233+
$('#chart-tooltip').show();
234+
236235
})
237236
.on('mouseout', function(d) {
238237
$scope.displayCoders = false
239238
$scope.highlightedRating = false
240-
$('#chart-tooltip').removeClass('distribution')
241-
$('#chart-tooltip').hide();
239+
$('#chart-tooltip').removeClass('distribution')
240+
$('#chart-tooltip').hide();
242241
$scope.$digest()
243242
})
244243

app/directives/history-graph/history-graph.directive.js

Lines changed: 32 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
import angular from 'angular'
22
import moment from 'moment'
33
import d3 from 'd3'
4-
import React from 'react'
54
import ReactDOM from 'react-dom'
65
import Tooltip from 'appirio-tech-react-components/components/Tooltip/Tooltip.jsx'
76

@@ -158,8 +157,8 @@ import Tooltip from 'appirio-tech-react-components/components/Tooltip/Tooltip.js
158157
var svg = d3.select('.history-graph').append('svg')
159158
.attr('width', w + padding.left + padding.right)
160159
.attr('height', h + padding.top + padding.bottom)
161-
162-
svg.append('rect')
160+
161+
svg.append('rect')
163162
.attr('x', padding.left)
164163
.attr('y', padding.top)
165164
.attr('width', w)
@@ -245,18 +244,18 @@ import Tooltip from 'appirio-tech-react-components/components/Tooltip/Tooltip.js
245244
return y
246245
}
247246
}
248-
ReactDOM.unmountComponentAtNode(document.getElementById('chart-tooltip'));
249-
ReactDOM.render(<Tooltip>
250-
<div className='tooltip-target'></div>
251-
<div className='tooltip-body'>
252-
<div className='tooltip-rating'></div>
253-
<div className='tooltip-challenge'>
254-
<div className='challenge-name'></div>
255-
<div className='challenge-date'></div>
256-
</div>
257-
</div>
258-
</Tooltip>
259-
, document.getElementById('chart-tooltip'));
247+
ReactDOM.unmountComponentAtNode(document.getElementById('chart-tooltip'));
248+
ReactDOM.render(<Tooltip>
249+
<div className='tooltip-target'></div>
250+
<div className='tooltip-body'>
251+
<div className='tooltip-rating'></div>
252+
<div className='tooltip-challenge'>
253+
<div className='challenge-name'></div>
254+
<div className='challenge-date'></div>
255+
</div>
256+
</div>
257+
</Tooltip>
258+
, document.getElementById('chart-tooltip'));
260259

261260
svg.selectAll('circle')
262261
.data(history)
@@ -275,31 +274,31 @@ import Tooltip from 'appirio-tech-react-components/components/Tooltip/Tooltip.js
275274
$scope.historyRating = d.newRating
276275
$scope.historyDate = moment(d.ratingDate).format('YYYY-MM-DD')
277276
$scope.historyChallenge = d.challengeName
278-
277+
279278
$scope.$digest()
280-
281-
d3.select('#chart-tooltip')
282-
.style('left', (d3.event.pageX-5) + 'px')
279+
280+
d3.select('#chart-tooltip')
281+
.style('left', (d3.event.pageX-5) + 'px')
283282
.style('top', (d3.event.pageY-5) + 'px')
284-
.style('display', 'block')
285-
d3.select('#chart-tooltip .tooltip-container')
286-
.style('left', '20px !important')
283+
.style('display', 'block')
284+
d3.select('#chart-tooltip .tooltip-container')
285+
.style('left', '20px !important')
287286
.style('top', '-20px !important')
288-
.style('opacity', '1')
289-
d3.select('#chart-tooltip .tooltip-container .tooltip-pointer')
290-
.style('left', '-5.5px !important')
287+
.style('opacity', '1')
288+
d3.select('#chart-tooltip .tooltip-container .tooltip-pointer')
289+
.style('left', '-5.5px !important')
291290
.style('bottom', '25px !important')
292-
293-
d3.select('#chart-tooltip .challenge-name').text($scope.historyChallenge);
294-
d3.select('#chart-tooltip .challenge-date').text(moment(d.ratingDate).format('MMM DD, YYYY'));
295-
d3.select('#chart-tooltip .tooltip-rating').text($scope.historyRating);
296-
d3.select('#chart-tooltip .tooltip-rating').style('background', ratingToColor($scope.colors, $scope.historyRating));
297-
291+
292+
d3.select('#chart-tooltip .challenge-name').text($scope.historyChallenge);
293+
d3.select('#chart-tooltip .challenge-date').text(moment(d.ratingDate).format('MMM DD, YYYY'));
294+
d3.select('#chart-tooltip .tooltip-rating').text($scope.historyRating);
295+
d3.select('#chart-tooltip .tooltip-rating').style('background', ratingToColor($scope.colors, $scope.historyRating));
296+
298297
})
299298
.on('mouseout', function(d) {
300299
$scope.historyRating = undefined
301-
$scope.$digest()
302-
})
300+
$scope.$digest()
301+
})
303302

304303
}
305304

0 commit comments

Comments
 (0)