Fixed bug parsing some inline attributes, and lined up the documentation for 4-space tabs.

Author: Chris Botman

dist/js/modaal.js

@@ -7,68 +7,68 @@
 	Modaal jQuery Plugin : Accessible Modals
 
 	==== General Options ===
-	type (string) 							    :	ajax, inline, image, iframe, confirm. Defaults to 'inline'
-	animation (string) 						  :	Fade, expand, down, up. Defaults to 'fade'
-	after_callback_delay (integer)	:	Specify a delay value for the after open callbacks. This is necessary because with the bundled animations
-																		have a set duration in the bundled CSS. Specify a delay of the same amount as the animation duration in so
-																		more accurately fire the after open/close callbacks. Defaults 350, does not apply if animation is 'none',
-																		after open callbacks are dispatched immediately
+	type (string) 					: ajax, inline, image, iframe, confirm. Defaults to 'inline'
+	animation (string) 				: Fade, expand, down, up. Defaults to 'fade'
+	after_callback_delay (integer)	: Specify a delay value for the after open callbacks. This is necessary because with the bundled animations
+										have a set duration in the bundled CSS. Specify a delay of the same amount as the animation duration in so
+										more accurately fire the after open/close callbacks. Defaults 350, does not apply if animation is 'none',
+										after open callbacks are dispatched immediately
 
-	is_locked (boolean)						  :	Set this to true to disable closing the modal via keypress or clicking the background. Beware that if
-																		type != 'confirm' there will be no interface to dismiss the modal if is_locked = true, you'd have to
-																		programmatically arrange to dismiss the modal. Confirm modals are always locked regardless of this option
-																		Defaults to false
+	is_locked (boolean)				: Set this to true to disable closing the modal via keypress or clicking the background. Beware that if
+										type != 'confirm' there will be no interface to dismiss the modal if is_locked = true, you'd have to
+										programmatically arrange to dismiss the modal. Confirm modals are always locked regardless of this option
+										Defaults to false
 
-	hide_close (boolean)					  :	Set this to true to hide the close modal button. Key press and overlay click will still close the modal.
-																		This method is best used when you want to put a custom close button inside the modal container space.
+	hide_close (boolean)			: Set this to true to hide the close modal button. Key press and overlay click will still close the modal.
+										This method is best used when you want to put a custom close button inside the modal container space.
 
-	background (string)						  : Background overlay style. Defaults to '#000'
-	overlay_opacity (float) 				: Background overlay transparency. Defaults to 0.8
-	overlay_close (boolean)					:	Set this to false if you want to disable click to close on overlay background.
+	background (string)				: Background overlay style. Defaults to '#000'
+	overlay_opacity (float) 		: Background overlay transparency. Defaults to 0.8
+	overlay_close (boolean)			: Set this to false if you want to disable click to close on overlay background.
 
-	accessible_title (string)				: Accessible title. Default 'Dialog Window'
-	start_open (boolean)					  :	Set this to true to launch the Modaal window immediately on page open
-	fullscreen (boolean)					  :	Set this to true to make the modaal fill the entire screen, false will default to own width/height attributes.
-	custom_class (string)					  :	Fill in this string with a custom class that will be applied to the outer most modal wrapper.
+	accessible_title (string)		: Accessible title. Default 'Dialog Window'
+	start_open (boolean)			: Set this to true to launch the Modaal window immediately on page open
+	fullscreen (boolean)			: Set this to true to make the modaal fill the entire screen, false will default to own width/height attributes.
+	custom_class (string)			: Fill in this string with a custom class that will be applied to the outer most modal wrapper.
 
-	width (integer)							    : Desired width of the modal. Required for iframe type. Defaults to undefined //TODO
-	height (integer)						    : Desired height of the modal. Required for iframe type. Defaults to undefined //TODO
+	width (integer)					: Desired width of the modal. Required for iframe type. Defaults to undefined //TODO
+	height (integer)				: Desired height of the modal. Required for iframe type. Defaults to undefined //TODO
 
 
 	=== Events ===
-	before_open (function) 					: Callback function executed before modal is opened
-	after_open (function)					  : Callback function executed after modal is opened
-	before_close (function)					: Callback function executed before modal is closed
-	after_close (function)					: Callback function executed after modal is closed
+	before_open (function) 			: Callback function executed before modal is opened
+	after_open (function)			: Callback function executed after modal is opened
+	before_close (function)			: Callback function executed before modal is closed
+	after_close (function)			: Callback function executed after modal is closed
 	source (function(element, src))	: Callback function executed on the default source, it is intended to transform the
-																		source (href in an AJAX modal or iframe). The function passes in the triggering element
-																		as well as the default source depending of the modal type. The default output of the
-																		function is an untransformed default source.
+										source (href in an AJAX modal or iframe). The function passes in the triggering element
+										as well as the default source depending of the modal type. The default output of the
+										function is an untransformed default source.
 
 
 	=== Confirm Options & Events ===
-	confirm_button_text (string)		: Text on the confirm button. Defaults to 'Confirm'
-	confirm_cancel_button_text (string):  Text on the confirm modal cancel button. Defaults to 'Cancel'
-	confirm_title (string)					: Title for confirm modal. Default 'Confirm Title'
-	confirm_content (string)				: HTML content for confirm message
-	confirm_callback (function)			: Callback function for when the confirm button is pressed as opposed to cancel
+	confirm_button_text (string)	: Text on the confirm button. Defaults to 'Confirm'
+	confirm_cancel_button_text (string) : Text on the confirm modal cancel button. Defaults to 'Cancel'
+	confirm_title (string)			: Title for confirm modal. Default 'Confirm Title'
+	confirm_content (string)		: HTML content for confirm message
+	confirm_callback (function)		: Callback function for when the confirm button is pressed as opposed to cancel
 
 
 	=== Gallery Options & Events ===
-	gallery_active_class (string)		:	Active class applied to the currently active image or image slide in a gallery 'gallery_active_item'
-	before_image_change (function)	:	Callback function executed before the image slide changes in a gallery modal. Default function( current_item, incoming_item )
-	after_image_change (function)		:	Callback function executed after the image slide changes in a gallery modal. Default function ( current_item )
+	gallery_active_class (string)	: Active class applied to the currently active image or image slide in a gallery 'gallery_active_item'
+	before_image_change (function)	: Callback function executed before the image slide changes in a gallery modal. Default function( current_item, incoming_item )
+	after_image_change (function)	: Callback function executed after the image slide changes in a gallery modal. Default function ( current_item )
 
 
 	=== AJAX Options & Events ===
-	loading_content (string)				:	HTML content for loading message. Default 'Loading …'
-	loading_class (string)					:	Class name to be applied while content is loaded via AJAX. Default 'is_loading'
-	ajax_error_class (string)				:	Class name to be applied when content has failed to load. Default is 'modaal-error'
-	ajax_success (function)		 			: Callback for when AJAX content is loaded in
+	loading_content (string)		: HTML content for loading message. Default 'Loading …'
+	loading_class (string)			: Class name to be applied while content is loaded via AJAX. Default 'is_loading'
+	ajax_error_class (string)		: Class name to be applied when content has failed to load. Default is 'modaal-error'
+	ajax_success (function)		 	: Callback for when AJAX content is loaded in
 
 
 	=== SOCIAL CONTENT ===
-	instagram_id (string)					  :	Unique photo ID for an Instagram photo.
+	instagram_id (string)			: Unique photo ID for an Instagram photo.
 
 */
 ( function( $ ) {
@@ -944,13 +944,13 @@
 				// option: after_callback_delay
 				if ( self.attr('data-modaal-locked') ) {
 					inline_options = true;
-					options.is_locked = (self.attr('data-modaal-locked') == 'true' || true ? true : false);
+					options.is_locked = (self.attr('data-modaal-locked') === 'true' ? true : false);
 				}
 
 				// option: hide_close
 				if ( self.attr('data-modaal-hide-close') ) {
 					inline_options = true;
-					options.hide_close = (self.attr('data-modaal-hide-close') == 'true' || true ? true : false);
+					options.hide_close = (self.attr('data-modaal-hide-close') === 'true' ? true : false);
 				}
 
 				// option: background
@@ -968,7 +968,7 @@
 				// option: overlay_close
 				if ( self.attr('data-modaal-overlay-close') ) {
 					inline_options = true;
-					options.overlay_close = (self.attr('data-modaal-overlay-close') == 'false' || false ? false : true);
+					options.overlay_close = (self.attr('data-modaal-overlay-close') === 'false' ? false : true);
 				}
 
 				// option: accessible_title
@@ -980,25 +980,25 @@
 				// option: start_open
 				if ( self.attr('data-modaal-start-open') ) {
 					inline_options = true;
-					options.start_open = (self.attr('data-modaal-start-open') == 'true' || true ? true : false);
+					options.start_open = (self.attr('data-modaal-start-open') === 'true' ? true : false);
 				}
 
 				// option: fullscreen
 				if ( self.attr('data-modaal-fullscreen') ) {
 					inline_options = true;
-					options.fullscreen = (self.attr('data-modaal-fullscreen') == 'true' || true ? true : false);
+					options.fullscreen = (self.attr('data-modaal-fullscreen') === 'true' ? true : false);
 				}
 
-				// option: fullscreen
+				// option: custom_class
 				if ( self.attr('data-modaal-custom-class') ) {
 					inline_options = true;
 					options.custom_class = self.attr('data-modaal-custom-class');
 				}
 
-				// option: fullscreen
+				// option: background_scroll
 				if ( self.attr('data-modaal-background-scroll') ) {
 					inline_options = true;
-					options.custom_class = (self.attr('data-modaal-background-scroll') == 'true' || true ? true : false);
+					options.background_scroll = (self.attr('data-modaal-background-scroll') === 'true' ? true : false);
 				}
 
 				// option: width
@@ -1115,7 +1115,7 @@
 		start_open: false,
 		fullscreen: false,
 		custom_class: '',
-		background_scroll: false,
+		background_scroll: false, // TODO: this property isn't described in the header
 
 		width: null,
 		height: null,