From 2f5125e8c48b47e741b3eee1cacb5c562512c051 Mon Sep 17 00:00:00 2001 From: MatmaRex Date: Wed, 4 Sep 2013 21:41:41 +0200 Subject: [PATCH] jquery.suggestions: Add some comments Change-Id: I0706e1f847f8ebf3d48cbb40a88dd6f6d487d2e6 --- resources/jquery/jquery.suggestions.js | 20 ++++++++------------ 1 file changed, 8 insertions(+), 12 deletions(-) diff --git a/resources/jquery/jquery.suggestions.js b/resources/jquery/jquery.suggestions.js index 93fba49412..bf38769021 100644 --- a/resources/jquery/jquery.suggestions.js +++ b/resources/jquery/jquery.suggestions.js @@ -379,8 +379,7 @@ $.suggestions = { preventDefault = false; switch ( key ) { - // Arrow down - case 40: + case 40: // Arrow down if ( wasVisible ) { $.suggestions.highlight( context, 'next', true ); context.data.selectedWithMouse = false; @@ -389,24 +388,21 @@ $.suggestions = { } preventDefault = true; break; - // Arrow up - case 38: + case 38: // Arrow up if ( wasVisible ) { $.suggestions.highlight( context, 'prev', true ); context.data.selectedWithMouse = false; } preventDefault = wasVisible; break; - // Escape - case 27: + case 27: // Escape context.data.$container.hide(); $.suggestions.restore( context ); $.suggestions.cancel( context ); context.data.$textbox.trigger( 'change' ); preventDefault = wasVisible; break; - // Enter - case 13: + case 13: // Enter context.data.$container.hide(); preventDefault = wasVisible; selected = context.data.$container.find( '.suggestions-result-current' ); @@ -585,13 +581,13 @@ $.fn.suggestions = function () { switch ( context.data.keypressed ) { // This preventDefault logic is duplicated from // $.suggestions.keypress(), which sucks - case 40: + case 40: // Arrow down e.preventDefault(); e.stopImmediatePropagation(); break; - case 38: - case 27: - case 13: + case 38: // Arrow up + case 27: // Escape + case 13: // Enter if ( context.data.$container.is( ':visible' ) ) { e.preventDefault(); e.stopImmediatePropagation(); -- 2.20.1