From 2d3170124dc289551144cc356873c374ced77f4c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Josef=20S=C3=B6ntgen?= Date: Tue, 29 Oct 2024 13:02:39 +0100 Subject: [PATCH] phone_manager: ignore AP list hovered temporarily The access-point-list hovered state prevents preseting the available scan results, which currently can happen unintentionally. This commit hot-wires the hover handling and thus will always show and update the list. New scan results are received every few seconds and it could happen that the list changes while the user selects a network (that's what the original implementation guards against). Issue #5369. --- repos/gems/src/app/phone_manager/main.cc | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/repos/gems/src/app/phone_manager/main.cc b/repos/gems/src/app/phone_manager/main.cc index a754205a45..087e4cc0d5 100644 --- a/repos/gems/src/app/phone_manager/main.cc +++ b/repos/gems/src/app/phone_manager/main.cc @@ -376,10 +376,11 @@ struct Sculpt::Main : Input_event_handler, */ bool ap_list_hovered() const override { - return _main_view.if_hovered([&] (Hovered_at const &at) { - return _network_widget.if_hovered(at, [&] (Hovered_at const &at) { - return _network_widget.hosted.if_hovered(at, [&] (Hovered_at const &at) { - return _network_widget.hosted.ap_list_hovered(at); }); }); }); + /* + * For now always report false so that scan-results + * will always be presented. + */ + return false; } /**