dashy
dashy copied to clipboard
[BUG] Scrolling in mobile constantly brings up edit menu on icons/items.
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.
- Open a dashy instance
- Start dev tools
- Switch to a mobile browser simulation with the device toolbar (ctrl + shift + M)
Please tick the boxes
- [X] You have explained the issue clearly, and included all relevant info
- [X] You are using a supported version of Dashy
- [X] You've checked that this issue hasn't already been raised
- [X] You've checked the docs and troubleshooting guide
- [X] You agree to the code of conduct
If you're enjoying Dashy, consider dropping us a ⭐
🤖 I'm a bot, and this message was automated
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.
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.
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.
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.
Issue is still relevant.
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.
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
Issue is still relevant.
still relevant
@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!