[Bug] Double-click to maximize sometimes triggered on right click.

[guyboots_thunderbroParticipant]

I have a mouse that has a problem (it sometimes triggers a double right-click). This causes NodeCanvas to crash Unity, because the maximize happens as a context-menu or node search is starting to show up and that makes Unity unhappy. It’s due to this line of code in GraphEditor.Events:

if ( e.type == EventType.MouseDown && e.clickCount == 2 )

It should be:

if ( e.type == EventType.MouseDown && e.clickCount == 2 && e.button == 0 )

to prevent any other double-clicks from triggering it.

[GavalakisKeymaster]

Hey,
Thank you for the fix 🙂
I’ve just made this change for the next version.

[Author]

Posts