Skip to content

Commit

Permalink
do not unset the ignoreNextMousedown option
Browse files Browse the repository at this point in the history
There is a bug in android where multiple mousedown events are being
fired so we don't want this being cleared after the first one. We will
set this callback up once and just make sure it's returning the correct
thing always.

There is only 1 use of this that I can find and it's in the exact spot that
we want to leave it around. We couldn't find any uses of it in classroom.
  • Loading branch information
mikehaverstock committed Sep 27, 2023
1 parent 4d4292e commit f72216d
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion src/services/mouse.ts
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,6 @@ class Controller_mouse extends Controller_latex {
(e.target as any).unselectable = true; // http://jsbin.com/yagekiji/1 // TODO - no idea what this unselectable property is

if (cursor.options.ignoreNextMousedown(e)) return;
else cursor.options.ignoreNextMousedown = ignoreNextMouseDownNoop;

// some elements should not act like internal mathquill nodes. Tokens for instance define external
// click / hover behaviors. So we have mathquill act like the item was never clicked. This allows
Expand Down

0 comments on commit f72216d

Please sign in to comment.