Improve slightly error handling for images [#116]

src/featherlight.gallery.js

@@ -107,7 +107,7 @@
 			return $.when(
 				self.getContent(),
 				$inner.fadeTo(self.galleryFadeOut,0.2)
-			).done(function($newContent) {
+			).always(function($newContent) {
 					self.setContent($newContent);
 					self.afterContent();
 					$newContent.fadeTo(self.galleryFadeIn,1);

src/featherlight.js

@@ -252,7 +252,7 @@
 					self.beforeContent(event);
 
 					/* Set content and show */
-					$.when($content).done(function($content){
+					$.when($content).always(function($content){
 						self.setContent($content);
 						self.afterContent(event);
 						/* Call afterOpen after fadeIn is done */
@@ -313,14 +313,14 @@
 				process: function(url)  {
 					var self = this,
 						deferred = $.Deferred(),
-						img = new Image();
+						img = new Image(),
+						$img = $('<img src="'+url+'" alt="" class="'+self.namespace+'-image" />');
 					img.onload  = function() {
-						var $img = $('<img src="'+url+'" alt="" class="'+self.namespace+'-image" />');
 						/* Store naturalWidth & height for IE8 */
 						$img.naturalWidth = img.width; $img.naturalHeight = img.height;
 						deferred.resolve( $img );
 					};
-					img.onerror = function() { deferred.reject(); };
+					img.onerror = function() { deferred.reject($img); };
 					img.src = url;
 					return deferred.promise();
 				}

test/featherlight_test.js

@@ -298,6 +298,16 @@ var stubAjaxLoad = function(content) {
 
 		});
 
+		describe('error handling', function() {
+			it('is quite basic', function(done) {
+				$.featherlight('does_not_exist.jpg');
+				patiently(done, function() {
+					expect($('.featherlight-image')).to.have.attr('src').equal('does_not_exist.jpg');
+					expect($('.featherlight')).not.to.have.class('featherlight-loading');
+				});
+			});
+		});
+
 		describe('iframe content filter', function() {
 			it('generates an iframe with the right attributes', function() {
 				$('<a data-featherlight-iframe="http://www.apple.com" '+