From 7628a6c855509f7e7378caf4ba1d1b5a5832bb97 Mon Sep 17 00:00:00 2001 From: webchick Date: Mon, 24 Mar 2014 07:55:57 -0700 Subject: [PATCH] Issue #1925492 by visabhishek, nod_, mkadin | YesCT: Small optimization for search on modules page. --- core/modules/system/system.modules.js | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/core/modules/system/system.modules.js b/core/modules/system/system.modules.js index 5ccad36c267..d65a7101d8b 100644 --- a/core/modules/system/system.modules.js +++ b/core/modules/system/system.modules.js @@ -17,7 +17,7 @@ var $input = $('input.table-filter-text').once('table-filter-text'); var $table = $($input.attr('data-table')); var $rowsAndDetails, $rows, $details; - + var searching = false; function hidePackageDetails(index, element) { var $details = $(element); var $visibleRows = $details.find('table:not(.sticky-header)').find('tbody tr:visible'); @@ -36,13 +36,15 @@ // Filter if the length of the query is at least 2 characters. if (query.length >= 2) { + searching = true; $rows.each(showModuleRow); // Hide the package
if they don't have any visible rows. // Note that we first show() all
to be able to use ':visible'. $details.show().each(hidePackageDetails); } - else { + else if (searching) { + searching = false; $rowsAndDetails.show(); } }