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 jaya-Adobe opened this issue 1 year ago • 4 comments

Code of conduct

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

Impacted component(s)

Firefly

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 Firefly web app in android. Encountered in Samsung s20+ and Samsung S21

Screenshots

https://github.com/adobe/spectrum-web-components/assets/110904672/b698645e-80c7-472a-8255-bae1c23b9220

What browsers are you seeing the problem in?

Chrome

How can we reproduce this issue?

  1. Go to https://firefly-stage.corp.adobe.com/upload/video-object-removal
  2. Click on Upload video file
  3. Upload a file and remove an object from it.
  4. Now go back on Upload page again you must see this Queue As in screen recording.
  5. Click on action-menu and see the behaviour.

Device: Samsung Galaxy S20+ 5g Chrome version - 124.0 Os - Android 13

Other way to reproduce, simply open this storybook on above device and click on action menu : https://studio.webcomponents.dev/edit/GnkodO5cuznEsnNm8DyF/src/index.ts?p=stories

NOTE: The problem does not reproduce in Dev tools/emulators. It happens on actual devices.

Sample code that illustrates the problem

Story book which reflects problem- https://studio.webcomponents.dev/edit/GnkodO5cuznEsnNm8DyF/src/index.ts?p=stories

Logs taken while reproducing problem

No response

jaya-Adobe avatar May 15 '24 16:05 jaya-Adobe

duplicate of #2575 This issue seems to have been fixed by this comment as informed by the chromium team. Can you please upgrade to the latest Android version and Chromium and test the bug! Feel free to let us know the results!

Rajdeepc avatar May 16 '24 04:05 Rajdeepc

@Rajdeepc Sure, but would there be no resolution for users on lower versions even if fixed for latest versions?

jaya-Adobe avatar May 16 '24 15:05 jaya-Adobe

If you go through the comment from the chromium team, its an issue on pointer trigger on how it parses on Android OS. As per them they have created a fix on the latest version of Android and its advisable to upgrade to the latest to bring in the patch!

Rajdeepc avatar May 16 '24 15:05 Rajdeepc

Be sure to checkout this comment, as well.

Westbrook avatar May 21 '24 17:05 Westbrook