adobe / spectrum-web-components / 27947187027

96%

push

github

fix(action-menu): propagate mobile-view to inner menu to prevent tray dismissal on submenu tap (#6425)

* fix(action-menu): propagate mobile-view to inner menu to prevent tray dismissal on submenu tap

On mobile, sp-action-menu renders its menu inside an sp-tray but was not
setting mobile-view on the inner sp-menu. Without mobile-view, submenu
items used flyout overlay logic whose click events bubbled up to the tray
and triggered tray dismissal instead of opening the submenu. This affected
both direct touch and VoiceOver double-tap on iOS.

Adding ?mobile-view=${this.isMobile.matches && !this.forcePopover} to the
inner sp-menu template switches submenu activation to the drill-down path
(openMobileSubmenu), which stops event propagation before it reaches the
tray. Also documents the mobile-view attribute and mobile-back-label in
the menu README, which had no mention of this behavior.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* chore: add changeset for action-menu submenu mobile tray fix

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* docs(menu): add overlay-trigger to mobile-view demo

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* docs(menu): add submenuMobileView story for reviewer verification

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Rajdeep Chandra <rajdeepc@adobe.com>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

6016 of 6459 branches covered (93.14%)

Branch coverage included in aggregate %.

1 of 1 new or added line in 1 file covered. (100.0%)

37902 of 39173 relevant lines covered (96.76%)

458.45 hits per line