From cf41f4a527fe6e912c23eec29167c8255307105a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marco=20Trevisan=20=28Trevi=C3=B1o=29?= Date: Wed, 17 Feb 2021 02:31:32 +0100 Subject: [PATCH] searchController: Get rid of activePage reference in key handling Even if activePage has been removed as part of commit 27627bd40, we've still a reference of it in key press handler. Given that there's no anymore an active page to redirect input to, remove these references, so that can be handled in the proper view to implement key-navigation. Part-of: --- js/ui/searchController.js | 8 -------- 1 file changed, 8 deletions(-) diff --git a/js/ui/searchController.js b/js/ui/searchController.js index 6fe76dafc..5d25897c7 100644 --- a/js/ui/searchController.js +++ b/js/ui/searchController.js @@ -151,14 +151,6 @@ var SearchController = GObject.registerClass({ return Clutter.EVENT_STOP; } else if (this._shouldTriggerSearch(symbol)) { this.startSearch(event); - } else if (!this._searchActive && !global.stage.key_focus) { - if (symbol === Clutter.KEY_Tab || symbol === Clutter.KEY_Down) { - this._activePage.navigate_focus(null, St.DirectionType.TAB_FORWARD, false); - return Clutter.EVENT_STOP; - } else if (symbol === Clutter.KEY_ISO_Left_Tab) { - this._activePage.navigate_focus(null, St.DirectionType.TAB_BACKWARD, false); - return Clutter.EVENT_STOP; - } } return Clutter.EVENT_PROPAGATE; }