Merge pull request #8 from thisissoon/feature/remove-isolated-scope

Fix: remove isolated scope from directive
thisissoon · Jun 13, 2016 · fc27952 · fc27952
2 parents e6d2d2f + b6a7b63
commit fc27952
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 4 deletions.
diff --git a/app/js/smartNav.js b/app/js/smartNav.js
@@ -37,7 +37,6 @@ angular.module('sn.smartNav', [
   function ($window, $document){
     return {
       restrict: 'A',
-      scope: {},
       link: function($scope, $element){
         /**
          * The last recorded scrollTop position

diff --git a/tests/unit/smartNav.js b/tests/unit/smartNav.js
@@ -2,7 +2,7 @@
 
 describe('sn.smartNav', function (){
 
-  var element, $scope, $rootScope, $document, $window, isolatedScope;
+  var element, $scope, $rootScope, $document, $window;
 
   beforeEach(module('sn.smartNav'));
 
@@ -27,8 +27,6 @@ describe('sn.smartNav', function (){
       element = $compile(element)($scope);
       $scope.$digest();
 
-      isolatedScope = element.isolateScope();
-
     }));
 
     afterEach(function(){