dashy icon indicating copy to clipboard operation
dashy copied to clipboard

[BUG] Scrolling in mobile constantly brings up edit menu on icons/items.

Open redge76 opened this issue 2 years ago • 10 comments

Environment

Self-Hosted (Docker)

System

Docker running on an Ubuntu system

Version

latest

Describe the problem

When scrolling up and down in mobile view you have to put your finger on the items or icons to scroll. When you do so it bring up the context menu for that item. This is extremely annoying and makes the dashboard painful to use because then I have to click to dismiss the context menu before I click an item on my dashboard. This is duplicate of #1135 (already closed)

Additional info

This issue can be easly reproduced using the dev tools in chrome.

  1. Open a dashy instance
  2. Start dev tools
  3. Switch to a mobile browser simulation with the device toolbar (ctrl + shift + M)

Please tick the boxes

redge76 avatar Jul 14 '23 17:07 redge76

If you're enjoying Dashy, consider dropping us a ⭐
🤖 I'm a bot, and this message was automated

liss-bot avatar Jul 14 '23 17:07 liss-bot

v1.9.2 running on https://live.dashy.to/ of https://demo.dashy.to/ is not impacted by this bug.
v2.1.1 running on https://dev.dashy.to/ is impacted.

I guess it is related to the longpress detection in file LongPress.js. It seems that this code does not take in account the possibility of a "touch/drag" gesture. It only check le lenght the screen is touched without verifying if the touch location has moved.

Sorry but I have no skill with nodejs and I can't correct this myself.

redge76 avatar Jul 18 '23 21:07 redge76

The best workaround I found is to disable the context menu altogether. edit mode -> edit app config -> Disable Context Menus These context menu are not mandatory and at least dashy is usable on a mobile.

redge76 avatar Aug 07 '23 23:08 redge76

I have this same issue. It would be nice if we could only disable the context menus on small screens that are more likely to have touch interfaces.

shall0pass avatar Aug 11 '23 23:08 shall0pass

This issue has gone 3 months without an update. To keep the ticket open, please indicate that it is still relevant in a comment below. Otherwise it will be closed in 5 working days.

liss-bot avatar Nov 10 '23 01:11 liss-bot

Issue is still relevant.

redge76 avatar Nov 10 '23 09:11 redge76

i solved the context issue by disabling it. like others pointed out, why not disable for small screens as a default? surely i am not the only one having scrolling issues cauzing pop ups. Sure the simple solution is to turn it off. but at least mention that in the setup guide for mobile users.

mooglestiltzkin avatar Nov 14 '23 16:11 mooglestiltzkin

Is it possible to disable the context menus for the mobile device only ? I've tried to "save locally" but when I refresh the window it re-enables the context menus again. Thank you

mainTAP avatar Nov 25 '23 12:11 mainTAP

Issue is still relevant.

OxYancy avatar Dec 10 '23 15:12 OxYancy

still relevant

CrazyWolf13 avatar Dec 28 '23 16:12 CrazyWolf13

@Lissy93, any chance you review and eventually merge the fix I provided? The issue is relly annoying on a mobile device 🙂 Thank you in advance!

ddenev avatar Jun 22 '24 07:06 ddenev