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

[4.3-beta1] Floating script editor breaks 3D viewport #93253

Closed
Flynsarmy opened this issue Jun 17, 2024 · 6 comments
Closed

[4.3-beta1] Floating script editor breaks 3D viewport #93253

Flynsarmy opened this issue Jun 17, 2024 · 6 comments

Comments

@Flynsarmy
Copy link
Contributor

Flynsarmy commented Jun 17, 2024

Tested versions

  • Reproducible in 4.3-beta1
  • NOT reproducible in 4.3-dev6

System information

Windows 11, geForce 4080, Godot 4.3-beta1, Forward+

Issue description

Making the script editor floating causes the 3D viewport to become unresponsive.

Steps to reproduce

  • Open any project
  • On the 'Script' tab click the 'Make the script editor floating.' button at top right to enable floating script editor
  • Move the script window onto a second monitor
  • Try to move in the 3D viewport back on the first monitor with your mouse. For example hold middle mouse button down and move your mouse.

It seems to be an input related issue. Framerate in the viewport remains perfectly fine. It's just mouse input that seems to not work anymore.

Minimal reproduction project (MRP)

N/A. Happens in any and all projects.

@smix8
Copy link
Contributor

smix8 commented Jun 17, 2024

Not reproducible for me with those steps on either beta1 or Godot master.

@Flynsarmy
Copy link
Contributor Author

I found a potential missing step. This problem only occurs if I move the script tab to a second monitor before trying to rotate in 3D viewport. I've updated the set of steps above.

I had someone on linux try and he couldn't replicate. Perhaps this is a Windows only bug? I'm on Win11 with and RTX 4080 geforce v555.99 drivers. Both monitors are 4K. Godot is on primary, script editor is moved to secondary.

@matheusmdx
Copy link
Contributor

This looks like another case of bug caused by pr #91361, @Flynsarmy can you check the comentary (#92954 (comment)) and do the steps to confirm?

@akien-mga
Copy link
Member

CC @bruvzg

@Flynsarmy
Copy link
Contributor Author

Flynsarmy commented Jun 17, 2024

I can confirm the issue is fixed in #92742. Since that ones already been merged are we good to close this issue?

@matheusmdx
Copy link
Contributor

I can confirm the issue is fixed in #92742. Since that ones already been merged are we good to close this issue?

Yep, no problem in close

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Release Blocker
Development

No branches or pull requests

5 participants