search

jonorthwash / ud-annotatrix

GNU General Public License v3.0
63 stars 49 forks source link

Firefox find-as-you-type blocks keyboard shortcuts #473

Open nschneid opened 2 years ago

nschneid commented 2 years ago

E.g. R to set the root, which appears to be handled at:

https://github.com/jonorthwash/ud-annotatrix/blob/415327b7f0392b78bb5056619bede37cad89c2a5/client/gui/keyboard.js#L315-L318

Is there a missing event.preventDefault()?

Is there alternative way to set the root?

nschneid commented 2 years ago

(A hacky workaround: app.graph.setRoot($(".activated")); in the console)

ftyers commented 2 years ago

@nschneid does event.preventDefault() work there as a fix? If so I can add it, or you could make a PR.

nschneid commented 2 years ago

I see it is being called in some places in that file. I don't know what the correct logic should be.