spectrum-web-components icon indicating copy to clipboard operation
spectrum-web-components copied to clipboard

[Bug]: No tray behavior with sp-action-menu on Android

Open HenryPing opened this issue 3 years ago • 2 comments

Code of conduct

  • [X] I agree to follow this project's code of conduct.

Impacted component(s)

ProjectX

Expected behavior

On Mobile, sp-action-menu is supposed to display its menu items within in tray from the bottom of the screen.

Actual behavior

No tray, instead, sp-action-menu display its menu items within in popover style on Android as it does on desktop. This happens in both ProjectX shell app on Android and Chrome browser on Android.

More details/discussion can be seen at this Slack thread: https://cclight.slack.com/archives/CESK60MQD/p1662762810680189

Screenshots

Screen Shot 2022-09-12 at 3 28 58 PM Screen Shot 2022-09-12 at 3 29 55 PM ![Uploading Screen Shot 2022-09-12 at 3.29.55 PM.png…]()

What browsers are you seeing the problem in?

Chrome

How can we reproduce this issue?

There are 2 ways:

  1. ProjectX on Android

    • Install and launch ProjectX shell app on Android device
    • open an existing document
    • tap the page adornment("... Page 1") at the top of the page
    • The page adornment menu items are shown as popover instead of tray, attached image 1
  2. Chrome on Android:

    • Open sp-action-menu example( https://opensource.adobe.com/spectrum-web-components/components/action-menu/ ) in Chrome
    • Tap the "... More Action"
    • The menu items are shown as popover instead of tray, attached image 2

Sample code that illustrates the problem

No response

Logs taken while reproducing problem

No response

HenryPing avatar Sep 12 '22 22:09 HenryPing

@HenryPing have you been able to confirm this state in the context of an actual device? I'd be interested in knowing if this is failing on pointer: coarse as it seems you're on an emulator, which would mean it could read you mouse as pointer: fine... 🤔 A quick test would be to be whether the docs site loads in medium or large scale by default in this context, as it also uses roughly the same media query.

Westbrook avatar Sep 13 '22 00:09 Westbrook

@Westbrook Yes, it is on the actual device. I used Vysor to mirror the device screen to my desktop to make it easier to take screen shot & video.

My android device info: Samsung S21 Android version: 12 Chrome version: 105.0.5195.79

HenryPing avatar Sep 13 '22 01:09 HenryPing

Shared this info to https://bugs.chromium.org/p/chromium/issues/detail?id=1384181

Westbrook avatar Jun 06 '23 03:06 Westbrook

It seems like the bug was addressed per this comment on Friday. Perhaps we can test the latest version of Chromium on Android and see if this is still reproducible? @HenryPing

najikahalsema avatar Jul 25 '23 21:07 najikahalsema

Chromium/Android has since shipped updates that should normalize this delivery context.

Westbrook avatar Dec 11 '23 15:12 Westbrook