Skip to content
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

Toggling distraction-free mode selects (or opens) a fixed script in the script list #90654

Closed
mieldepoche opened this issue Apr 14, 2024 · 3 comments · Fixed by #90816 or #91168
Closed

Comments

@mieldepoche
Copy link
Contributor

Tested versions

  • does not happen in 4.3 dev 5
  • happens in v4.3.dev.custom_build [578d937]

System information

linux

Issue description

  • when toggling distraction free mode, a previously opened script is automatically re-opened

    • it happens independently of the number of opened scripts
  • if deleted, it reopens anyway as an unsaved script

  • in the clip below, I create a new script and toggle distraction free mode after closing the script: notice how it gets automatically re-openened. Note that I use a shortcut to close the script, which is why it happens quickly.

simplescreenrecorder-2024-04-14_12.47.00.mp4

Steps to reproduce

  • create a script and open it, and close it
  • toggle distraction free mode
  • notice how the script opens automatically

Minimal reproduction project (MRP)

any

@kitbdev
Copy link
Contributor

kitbdev commented Apr 15, 2024

It unified all methods that affect docks, so toggling distraction free mode now calls EditorNode::get_singleton()->edit_current();, which causes this behavior. Before, some methods called edit_current and some didn't, so I'm not sure if it's necessary.

So, should changing docks not call edit_current or should edit_current be changed?

@KoBeWi
Copy link
Member

KoBeWi commented Apr 17, 2024

Related:

godot.windows.editor.dev.x86_64_damBvJ8CB5.mp4

After you created a script, clicking any dock will open script editor. Extremely annoying.

@akien-mga
Copy link
Member

Reopening as #90816 was reverted due to a crash.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Bad
4 participants