From a980f5b0d08e86c26eeafc4e41ac300cae03ff18 Mon Sep 17 00:00:00 2001 From: Andreas Reinhold / reini Date: Sun, 13 Sep 2026 22:41:51 +0200 Subject: [PATCH] Expect the section picker to mark its page, not a checked item The browser test still looked for the menuitemcheckbox the picker drew before it marked the current section with aria-current. Co-Authored-By: Claude Opus 5 (1M context) Claude-Session: https://claude.ai/code/session_01RHoXZSHc8gGpZjFmA5fPc2 --- tests/Browser/BarsTest.php | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/tests/Browser/BarsTest.php b/tests/Browser/BarsTest.php index 976c76ca..61d70653 100644 --- a/tests/Browser/BarsTest.php +++ b/tests/Browser/BarsTest.php @@ -141,6 +141,7 @@ it('turns section tabs into a picker on a phone', function () { ->assertScript("getComputedStyle(document.querySelector('[data-section-nav] nav')).display === 'none'") ->click('[data-section-picker] button') ->assertScript("document.querySelector('[data-section-picker] [popover]').matches(':popover-open')") - ->assertAttribute('[data-section-picker] [role="menuitemcheckbox"][href="#profile"]', 'aria-checked', 'true') - ->assertAttribute('[data-section-picker] [role="menuitemcheckbox"][href="#security"]', 'aria-checked', 'false'); + // A menu of places: the current section is the page, not a checked choice. + ->assertAttribute('[data-section-picker] [role="menuitem"][href="#profile"]', 'aria-current', 'page') + ->assertScript("! document.querySelector('[data-section-picker] [role=\"menuitem\"][href=\"#security\"]').hasAttribute('aria-current')"); });