r13458 - Products.Archetypes/trunk/Products/Archetypes/skins/archetypes/widgets/js

Author: rmattb
Date: Sat Feb  5 04:18:08 2011
New Revision: 13458

Modified:
   Products.Archetypes/trunk/Products/Archetypes/skins/archetypes/widgets/js/keywordmultiselect.js
Log:
Improved comments in JavaScript for keyword multiple select enhancement PLIP. Refs #11017.

Modified: Products.Archetypes/trunk/Products/Archetypes/skins/archetypes/widgets/js/keywordmultiselect.js
==============================================================================
--- Products.Archetypes/trunk/Products/Archetypes/skins/archetypes/widgets/js/keywordmultiselect.js	(original)
+++ Products.Archetypes/trunk/Products/Archetypes/skins/archetypes/widgets/js/keywordmultiselect.js	Sat Feb  5 04:18:08 2011
@@ -111,6 +111,7 @@
 		// Initialize selected options list
 		updateSelected.call(multiSelectA);
 		
+		// --- Navigation with Mouse ---
 		// Handle mouse hover of option, both
 		// entering an option, *and* 
 		// mouse moving within an option.
@@ -135,6 +136,7 @@
 			}
 		});
 		
+		// --- Selection with Mouse ---
 		// Handle mouse click of checkbox
 		multiSelectOptions.find('INPUT:checkbox').click( function() {
 			// set the label checked class
@@ -150,6 +152,7 @@
 			}
 		});
 		
+		// --- Navigation with Tab Key ---
 		// Track mouse click of option
 		multiSelectOptions.find('LABEL').mousedown(function(){
 			// Track mouse clicks, 
@@ -180,6 +183,7 @@
 		
 			var multiSelectOptions = $(this).next('.multiSelectOptions');
 			
+			// --- Navigation with Arrow or Page Keys ---
 			// Down || Up
 			if( e.keyCode == 40 || e.keyCode == 38) {
 				var allOptions = multiSelectOptions.find('LABEL');
@@ -213,7 +217,6 @@
 				
 				return false;
 			}
-
 			// Page up || Page down
 			if( e.keyCode == 33 || e.keyCode == 34) {
 				var allOptions = multiSelectOptions.find('LABEL');
@@ -241,6 +244,7 @@
 				return false;
 			}
 				
+			// --- Selection with Keyboard ---
 			// Enter, Space
 			if( e.keyCode == 13 || e.keyCode == 32 ) {
 				var selectedCheckbox = multiSelectOptions.find('LABEL.hover INPUT:checkbox');

------------------------------------------------------------------------------
The modern datacenter depends on network connectivity to access resources
and provide services. The best practices for maximizing a physical server's
connectivity to a physical network are well understood - see how these
rules translate into the virtual world? 
http://p.sf.net/sfu/oracle-sfdevnlfb