A solution that automatically switch to the script tab.#23
A solution that automatically switch to the script tab.#23RedMser merged 1 commit intoRedMser:masterfrom
Conversation
RedMser
left a comment
There was a problem hiding this comment.
Thanks for the PR!
Overriding _handles in general seems fine to me. But I'm unsure if calling set_main_screen_editor here is the best approach, since _handles is meant to be a pure function to find out info, but it should not be changing anything.
E.g. _handles it is called in the FileSystem tooltip code, so does this mean that hovering a script file in the file system dock now opens the script editor?
I assume the call you're looking for comes from get_handling_sub_editors here. Maybe the code below this, in this same function, has some better extension points? Like maybe you can also override _edit or _make_visible of the plugin, and move the main screen switch to there.
|
Hi ! , what happened with me is that execution only occurs if the script file is opened. |
|
RedMser
left a comment
There was a problem hiding this comment.
Alright... in that case I'll accept since it fixes the problem and doesn't seem to cause other problems.
Thanks again ^^
|
Thank you, and best wishes .! |
Finally, it was simple ! 😅.