Bug ShortcutManager wrong Event.current.mouse.position from Shortcut attribute

Let me explain better than the horrific title.
The following Editor class log on console the current mouse position .
The problem is that the same function is called creating a shortcut using the Shortcut attribute, and the result is that if i press the key, the two pieces of code print different mouse positions.

Code (CSharp):

1. using UnityEngine;
2. using UnityEditor;
3. using UnityEditor.ShortcutManagement;
4.  
5. [CustomEditor(typeof(TestScript), true)]
6. public class TestScriptEditor : Editor
7. {
8.     protected void OnSceneGUI()
9.     {
10.         Event e = Event.current;
11.         switch (e.type)
12.         {
13.             case EventType.KeyDown:
14.             {
15.                 if (Event.current.keyCode == (KeyCode.Alpha0))
16.                     Debug.Log("mousepos OnSceneGUI= " + Event.current.mousePosition.ToString());
17.                 //e.Use();
18.                 break;
19.             }
20.         }
21.     }
22.  
23.     [Shortcut("test/shortcutTest2", KeyCode.Alpha0)]
24.     public static void ToolWindowToggleAlternateMode(ShortcutArguments args)
25.     {
26.         Vector2 mousePos = Event.current.mousePosition;
27.         Debug.Log("mousepos from SHORTCUT MANAGER= " + Event.current.mousePosition.ToString());
28.     }
29. }
30.  

Seems that the one called as shortcut is getting the wrong mouse position, but it is the one that i need to use, is that any way to fix this?
Or would be better if there is any way to bind shortcuts and actions using the ShortcutManager.instance

I attach two screenshot:
- the one is the result of 3 keydown with mouse in different position, as you can see the one called on shortcut always has the same difference.
- and the other is Event.current state debugged in visual studio, the left one is called in OnSceneGUI and the right one is called from shortcut.


Thank you in advance.

 

Hi Tomas, thank you for helping me.

you give me a good solution but let me extend my problem.

I have a Shortcut manager Class Like this:

Code (CSharp):

1.  
2.  
3. [ExecuteInEditMode, UnityEditor.InitializeOnLoad]
4.  
5. public class MyShortcutManager : MonoBehaviour{
6.  
7. public static MyShortcutManager Instance;
8. private Dictionary<string, Dictionary<KeyCode keyCode, UnityEvent>> m_shortcutsActionsDictionary;
9.  
10. void AddListener(KeyCode keyCode, UnityEvent event){// bind action to key and put in dictionary}
11.  
12. void TriggerEvent(){// get and call UnityEvent from dictionary}
13.  
14. [Shortcut("test/TriggerW", KeyCode.W)]
15.     public static void TriggerW(ShortcutArguments args)
16.     {
17.       TriggerEvent(args, KeyCode.W);
18.     }
19.  
20. [Shortcut("test/TriggerA", KeyCode.A)]
21.     public static void TriggerW(ShortcutArguments args)
22.     {
23.       TriggerEvent(args, KeyCode.A);
24.     }
25. }
26.  
27.  

NB: I need to use shortcutManager to create and switch profiles between windows ad because it is a constraint of the task)

This is another simplification but just for this example could work.

So i attach this to a object and then other teams member register my actions from other script to keyPress , the actions can be use a function that is used also onGui and get the mouse position to find click point, i cant know if they are calling that from onGui or ShortcutManager and i cannot hardcode some variables to know where i came from.

I think the big problem is the method used to create shortcut (with attribute) seems to lose context. If i can create shortcut by a simple shortcut manager function call maybe i can avoid context switch.

Do you know if there is a way to bind an action to a shortcut without the use of Attributes? Maybe something Like ShortcutManager.bindAction(shortcut, action).

I need to find a solution to adapt my shortcut manager to the rest of the program.