r13292 - Products.Archetypes/branches/plip11017-tag-selector-rmattb/Products/Archetypes/skins/archetypes/widgets/js

Author: rmattb
Date: Thu Nov 25 08:15:59 2010
New Revision: 13292

Modified:
   Products.Archetypes/branches/plip11017-tag-selector-rmattb/Products/Archetypes/skins/archetypes/widgets/js/keywordmultiselect.js
Log:
Improve comment on tab key handling.  Refs #11017.

Modified: Products.Archetypes/branches/plip11017-tag-selector-rmattb/Products/Archetypes/skins/archetypes/widgets/js/keywordmultiselect.js
==============================================================================
--- Products.Archetypes/branches/plip11017-tag-selector-rmattb/Products/Archetypes/skins/archetypes/widgets/js/keywordmultiselect.js	(original)
+++ Products.Archetypes/branches/plip11017-tag-selector-rmattb/Products/Archetypes/skins/archetypes/widgets/js/keywordmultiselect.js	Thu Nov 25 08:15:59 2010
@@ -269,12 +269,8 @@
 				// Dropdown is visible
 				
 				// Tab
-				// When tabbing, none of this key-detecting code is currently never invoked. 
-				// Even when tabbing from before the selector, all the way through every option, 
-				// and beyond into the rest of the page. 
-				// With the removal of the a.multiSelect from the normal tabindex 
-				// (not needed since this box is always dropped-down), 
-				// the multiSelect is never touched, so this code is never invoked.
+				// When tabbing, this key-detecting code is currently never invoked,
+				// because tabbing gives focus to the checkbox, not the multiSelect. 
 				if( e.keyCode == 9 ) {
 					$(this).addClass('focus').trigger('click'); // esc, left, right - hide
 					$(this).focus().next(':input').focus();

------------------------------------------------------------------------------
Increase Visibility of Your 3D Game App & Earn a Chance To Win $500!
Tap into the largest installed PC base & get more eyes on your game by
optimizing for Intel(R) Graphics Technology. Get started today with the
Intel(R) Software Partner Program. Five $500 cash prizes are up for grabs.
http://p.sf.net/sfu/intelisp-dev2dev