You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Whilst looking through the codebase's event handling logic I found this property that doesn't seem to be used or documented. Here is the Github search result:
As you can see from the image above, it's only referenced four times in the codebase, and only once in a file that isn't for testing.
I may be mistaken but it doesn't actually seem to be used for anything as it's replaced by keyIsDown which is documented.
This brings up the question, is there any point keeping track of this variable and should it be removed (I think it should as there's no point keeping it).
The text was updated successfully, but these errors were encountered:
Topic
Whilst looking through the codebase's event handling logic I found this property that doesn't seem to be used or documented. Here is the Github search result:
As you can see from the image above, it's only referenced four times in the codebase, and only once in a file that isn't for testing.
I may be mistaken but it doesn't actually seem to be used for anything as it's replaced by
keyIsDown
which is documented.This brings up the question, is there any point keeping track of this variable and should it be removed (I think it should as there's no point keeping it).
The text was updated successfully, but these errors were encountered: