-
-
Notifications
You must be signed in to change notification settings - Fork 21k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix out of sync ScriptEditor distraction mode when Separate Distraction Mode
is Enabled
#48834
Fix out of sync ScriptEditor distraction mode when Separate Distraction Mode
is Enabled
#48834
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure if this is the correct fix, it looks like _edit_current
should call _editor_select
rather then it re-implementing the switching code.
@Paulb23 I have had a look at those two functions and can see what you mean. Thanks for putting me in the right direction. I will rework this to be as you suggest. |
3d2932c
to
7cefacd
Compare
@Paulb23 I have updated this. Let me know if I have missed anything. |
7cefacd
to
a6b044e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the contribution!
LGTM.
a6b044e
to
45fc6c4
Compare
Thanks! |
Cherry-picked for 3.4. |
Fixes an issue where the Script Editor's distraction state can get out of sync.
This happens when
Separate Distraction Mode
is enabled, distraction-free mode is active on the Script Editor, the user switches to a 2D or 3D Scene Editor and clicks a SceneTree Node's Edit Script button.Click to see the issue in action