Merge pull request #20 from lifter035/main

Gabriel Delépine · Gabriel Delépine · commit 19a5272776e6 · 2014-12-18T10:21:32.000+01:00
Scope retrieved from $rootElement instead of &lt;head&gt;
diff --git a/angular-css-injector.js b/angular-css-injector.js
@@ -10,7 +10,7 @@ angular.module('angular.css.injector', [])
 .provider('cssInjector', function() {
 	var singlePageMode = false;
 
-	function CssInjector($compile, $rootScope){
+	function CssInjector($compile, $rootScope, $rootElement){
         // Variables
         var head = angular.element(document.getElementsByTagName('head')[0]),
             scope;
@@ -27,8 +27,7 @@ angular.module('angular.css.injector', [])
         {
             if(scope === undefined)
             {
-                if((scope = head.scope()) === undefined) // We initialise head's scope in a separated function because it is not defined at the time of the initialisation of the service.
-                    throw("angular.css.injector error : Please initialize your app in the HTML tag and be sure your page has a HEAD tag.");
+                scope = $rootElement.scope();
             }
         };
 
@@ -83,8 +82,8 @@ angular.module('angular.css.injector', [])
         };
 	}
 
-	this.$get = ['$compile', '$rootScope', function($compile, $rootScope){
-		return new CssInjector($compile, $rootScope);
+	this.$get = ['$compile', '$rootScope', '$rootElement', function($compile, $rootScope, $rootElement){
+		return new CssInjector($compile, $rootScope, $rootElement);
 	}];
 
 	this.setSinglePageMode = function(mode){

Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@ angular.module('angular.css.injector', [])`
`10`	`10`	`.provider('cssInjector', function() {`
`11`	`11`	`var singlePageMode = false;`
`12`	`12`
`13`		`- function CssInjector($compile, $rootScope){`
	`13`	`+ function CssInjector($compile, $rootScope, $rootElement){`
`14`	`14`	`// Variables`
`15`	`15`	`var head = angular.element(document.getElementsByTagName('head')[0]),`
`16`	`16`	`scope;`
`@@ -27,8 +27,7 @@ angular.module('angular.css.injector', [])`
`27`	`27`	`{`
`28`	`28`	`if(scope === undefined)`
`29`	`29`	`{`
`30`		`- if((scope = head.scope()) === undefined) // We initialise head's scope in a separated function because it is not defined at the time of the initialisation of the service.`
`31`		`- throw("angular.css.injector error : Please initialize your app in the HTML tag and be sure your page has a HEAD tag.");`
	`30`	`+ scope = $rootElement.scope();`
`32`	`31`	`}`
`33`	`32`	`};`
`34`	`33`
`@@ -83,8 +82,8 @@ angular.module('angular.css.injector', [])`
`83`	`82`	`};`
`84`	`83`	`}`
`85`	`84`
`86`		`- this.$get = ['$compile', '$rootScope', function($compile, $rootScope){`
`87`		`- return new CssInjector($compile, $rootScope);`
	`85`	`+ this.$get = ['$compile', '$rootScope', '$rootElement', function($compile, $rootScope, $rootElement){`
	`86`	`+ return new CssInjector($compile, $rootScope, $rootElement);`
`88`	`87`	`}];`
`89`	`88`
`90`	`89`	`this.setSinglePageMode = function(mode){`