From 5ffa224041884583e130a55c82465c69390e2e3e Mon Sep 17 00:00:00 2001 From: Darren Burns Date: Thu, 24 Oct 2024 14:25:31 +0100 Subject: [PATCH] Optimise optionlist/radio_set scrolling method --- src/textual/_widget_navigation.py | 2 +- tests/test_widget_navigation.py | 6 ------ 2 files changed, 1 insertion(+), 7 deletions(-) diff --git a/src/textual/_widget_navigation.py b/src/textual/_widget_navigation.py index 1d010a17b..a5e829900 100644 --- a/src/textual/_widget_navigation.py +++ b/src/textual/_widget_navigation.py @@ -140,7 +140,7 @@ def find_next_enabled( for index, candidate in loop_from_index(candidates, anchor, direction, wrap=True): if not candidate.disabled: return index - return None + return anchor def find_next_enabled_no_wrap( diff --git a/tests/test_widget_navigation.py b/tests/test_widget_navigation.py index a322f3846..44f8b1715 100644 --- a/tests/test_widget_navigation.py +++ b/tests/test_widget_navigation.py @@ -142,16 +142,10 @@ def test_find_next_enabled_no_wrap(candidates, anchor, direction, result): @pytest.mark.parametrize( ["function", "start", "direction"], [ - (find_next_enabled, 0, 1), - (find_next_enabled, 0, -1), (find_next_enabled_no_wrap, 0, 1), (find_next_enabled_no_wrap, 0, -1), - (find_next_enabled, 1, 1), - (find_next_enabled, 1, -1), (find_next_enabled_no_wrap, 1, 1), (find_next_enabled_no_wrap, 1, -1), - (find_next_enabled, 2, 1), - (find_next_enabled, 2, -1), (find_next_enabled_no_wrap, 2, 1), (find_next_enabled_no_wrap, 2, -1), ],