search

riverscuomo / public-bug-hunt

A place to file bug reports for all of my apps
GNU General Public License v3.0
7 stars 0 forks source link

Long press on playlist has no effect #47

Closed SlightlyCompletely closed 1 year ago

SlightlyCompletely commented 1 year ago

The Issue A long press on any playlist in the library has no effect.

To Reproduce

  1. Navigate to Library.
  2. Long press on any playlist.
  3. Note there is no effect.

Expected behavior Long press on any playlist in the library should result in popup menu and share link.

Device

KCreek1 commented 1 year ago

Given the success I had scrolling on iPad with Bluetooth mouse I thought I'd go back and try something with this.

Using right click on mouse I was able to bring up the pop up (share) menu for tracks.

iPad 7th gen iPad is 16.7 Weezify 1.9.0 (171)

KCreek1 commented 1 year ago

This issue seems to be related to issue #14 in that this function works when using a mouse and not touch in iPad app and iPad web.

KCreek1 commented 1 year ago

I just tried this on Chromebook browser with touch screen. Same results.

Maybe for this action on large screen - touch is not listed as a pointer device kind?

https://stackoverflow.com/q/69253697

riverscuomo commented 1 year ago

something in TappablePlaylistWidget

i hope i didn't screw up onSecondaryTapUp: (details) => contextMenuBehavior,

riverscuomo commented 1 year ago

// The problem is that both TapUpDetails and LongPressEndDetails classes //do not inherit from the same superclass, so it's not possible to just use //a superclass type for the details parameter.

// A solution could be to define two separate methods for createWeezifyContextMenuBehavior. //Each one dedicated to a specific details type.

weezifymenus.dart

SlightlyCompletely commented 1 year ago

Resolved in Weezify 1.12.0 (186) on iPad Air running iPadOS 17.0.3.