From 96a2747e0b8347b70703052fc1c07db980d097bc Mon Sep 17 00:00:00 2001 From: Colin Walters Date: Thu, 2 Jul 2009 01:02:46 -0400 Subject: [PATCH] Wire up search up/down This is not a complete fix; the search selection will not cross the apps/docs boundary. But it's good enough until we have a more final search design. --- js/ui/overlay.js | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/js/ui/overlay.js b/js/ui/overlay.js index 478719fe9..e16423e2c 100644 --- a/js/ui/overlay.js +++ b/js/ui/overlay.js @@ -314,9 +314,19 @@ Dash.prototype = { // too, but there doesn't seem to be any flickering if we first select // something in one display, but then unset the selection, and move // it to the other display, so it's ok to do that. - // TODO: add the right logic + if (me._resultsDocsSection.display.hasSelected()) + me._resultsDocsSection.display.selectUp(); + else if (me._resultsAppsSection.display.hasItems()) + me._resultsAppsSection.display.selectUp(); + else + me._resultsDocsSection.display.selectUp(); } else if (symbol == Clutter.Down) { - // TODO: add the right logic + if (me._resultsDocsSection.display.hasSelected()) + me._resultsDocsSection.display.selectDown(); + else if (me._resultsAppsSection.display.hasItems()) + me._resultsAppsSection.display.selectDown(); + else + me._resultsDocsSection.display.selectDown(); } return false; });