safebox/guacamole-client
4
0
Fork 0
mirror of https://github.com/gyurix1968/guacamole-client.git synced 2025-09-06 13:17:41 +00:00
Code Issues Packages Projects Releases Wiki Activity

GUAC-587: Fix gratuitous $digest() from unnecessary $interval use.

Browse Source
This commit is contained in:
Michael Jumper
2015-04-30 15:28:07 -07:00
parent 0ee15eb474
commit bebde4720c
1 changed files with 9 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
guacamole/src/main/webapp/app/client/directives/guacViewport.js
View File

@@ -31,8 +31,12 @@ angular.module('client').directive('guacViewport', [function guacViewport() {
scope: {},
transclude: true,
templateUrl: 'app/client/templates/guacViewport.html',
controller: ['$scope', '$interval', '$window', '$document', '$element',
function guacViewportController($scope, $interval, $window, $document, $element) {
controller: ['$scope', '$injector', '$element',
function guacViewportController($scope, $injector, $element) {
// Required services
var $window = $injector.get('$window');
var $document = $injector.get('$document');
/**
* The fullscreen container element.
@@ -94,14 +98,14 @@ angular.module('client').directive('guacViewport', [function guacViewport() {
$window.addEventListener('scroll', fitVisibleArea);
// Poll every 10ms, in case scroll event does not fire
var pollArea = $interval(fitVisibleArea, 10);
var pollArea = $window.setInterval(fitVisibleArea, 10);
// Clean up on destruction
$scope.$on('$destroy', function destroyViewport() {
$window.removeEventListener('scroll', fitVisibleArea);
$interval.cancel(pollArea);
$window.clearInterval(pollArea);
});
}]
};
}]);
}]);