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

Author: rmattb
Date: Thu Feb  3 07:12:14 2011
New Revision: 13450

Modified:
   Products.Archetypes/trunk/Products/Archetypes/skins/archetypes/widgets/js/keywordmultiselect.js
Log:
Further qualified label selection in JavaScript in 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	Thu Feb  3 07:12:14 2011
@@ -111,7 +111,7 @@
 		// Handle all checkboxes
 		multiSelectOptions.find('INPUT:checkbox').click( function() {
 			// set the label checked class
-			$(this).parent("LABEL").toggleClass('checked', $(this).attr('checked'));
+			$(this).parent('LABEL').toggleClass('checked', $(this).attr('checked'));
 			
 			updateSelected.call(multiSelectA);
 			multiSelectA.focus();
@@ -125,7 +125,7 @@
 		
 		// Initial display
 		multiSelectOptions.each( function() {
-			$(this).find('INPUT:checked').parent().addClass('checked');
+			$(this).find('INPUT:checked').parent('LABEL').addClass('checked');
 		});
 		
 		// Initialize selected
@@ -169,8 +169,9 @@
 				// Without this, both moused & tabbed checks return focus to multiSelectA, 
 				// causing tabbed checkboxes to lose focus.
 				lastNavTabKeyCheckbox = $(this);
+				// Remove hover class from all labels in the div
 				lastNavTabKeyCheckbox.parent().parent().parent().find('LABEL').removeClass('hover');
-				lastNavTabKeyCheckbox.parent().addClass('hover');
+				lastNavTabKeyCheckbox.parent('LABEL').addClass('hover');
 			}
 			lastNavClickTag = null;
 		});
@@ -246,7 +247,7 @@
 				var selectedCheckbox = multiSelectOptions.find('LABEL.hover INPUT:checkbox');
 				
 				// Set the checkbox (and label class)
-				selectedCheckbox.attr('checked', !selectedCheckbox.attr('checked')).parent("LABEL").toggleClass('checked', selectedCheckbox.attr('checked'));
+				selectedCheckbox.attr('checked', !selectedCheckbox.attr('checked')).parent('LABEL').toggleClass('checked', selectedCheckbox.attr('checked'));
 				
 				updateSelected.call(multiSelectA);
 				return false;

------------------------------------------------------------------------------
Special Offer-- Download ArcSight Logger for FREE (a $49 USD value)!
Finally, a world-class log management solution at an even better price-free!
Download using promo code Free_Logger_4_Dev2Dev. Offer expires 
February 28th, so secure your free ArcSight Logger TODAY! 
http://p.sf.net/sfu/arcsight-sfd2d