OrbitControls: Add stopListenToKeyEvents().

[nikolas]

If I called listenToKeyEvents(), sometimes I want to remove this event listener, temporarily or otherwise.

[WestLangley]

"unlisten" is not a word. Is there a better API? 🤔

Also, maybe set this._domElementKeyEvents = null; on .dispose().

[Mugen87]

"unlisten" is not a word. Is there a better API?

from https://ell.stackexchange.com/questions/214085/is-there-any-word-for-unlisten-negative-of-listen

In the context of programming, it's better to be clear than to try and force "correct" English.

TBH, I feel very strong about this statement.

[Mugen87]

Also, maybe set this._domElementKeyEvents = null; on .dispose().

Good catch! I'll update the PR.

[WestLangley]

In the context of programming, it's better to be clear than to try and force "correct" English.

TBH, I feel very strong about this statement.

That's fine. But if you are using "unlisten" to mean "stop listening", then use stopListeningToKeyEvents.

That's both correct and clear.

Or stopListenKeyEvents.

[Mugen87]

How about stopListenToKeyEvents()? I think it's better for programming when the listenToKeyEvents() term is the same for both methods.

[WestLangley]

Oh yes. (That was a typo.)

[mrdoob]

"unlisten" is not a word. Is there a better API? 🤔

Can we make it a word? 😅

How about ignoreKeyEvents()?

listenKeyEvents()
ignoreKeyEvents()

I don't like that "To" in the name... 😕

[Mugen87]

The *to* names were discussed here: #21054 (comment).

I'm okay with the shorter version as well :)