search

1j01 / jspaint

🎨 Classic MS Paint, ＲＥＶＩＶＥＤ + ✨Extras
https://jspaint.app/about
MIT License
7.22k stars 562 forks source link

Fix Mobile Menu #154

Closed hancanban closed 2 years ago

hancanban commented 4 years ago

On Android When You Tap On A Menu It Selects And Then Unselects.

1j01 commented 4 years ago

Yeah, this is a long-standing bug. This has been on the TODO for... let me check, yeah literally years, haha. Since 2016. (via git blame)

There's a bunch of things I want to change/improve about the menus, so I've been thinking I want to rewrite that code entirely. I want to:

  1. Not use jQuery, so I can package the menus up with no dependencies for other projects to use
  2. Support context menus
  3. Not take focus when interacting with the mouse, so that Edit > Copy/Paste can work with textboxes
  4. Support more scenarios where the menus meet the bounds of the screen (like if there's just a really long menu, that can't fit on screen with repositioning, it should scroll).
  5. Work with touch as a primary concern, instead of tacked-on support. Sub-menus are also very hard to use on mobile right now.
  6. Keyboard navigation of sub-menus
  7. Screen reader support
1j01 commented 2 years ago

This is fixed.