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

Project manager erases project name after browsing for project path when creating a new project #95038

Closed
mattsullentrup opened this issue Aug 1, 2024 · 2 comments · Fixed by #95062

Comments

@mattsullentrup
Copy link

Tested versions

  • Reproducible in 4.3rc1 and rc2.
  • Not reproducible in 4.3beta3 and earlier.

System information

Godot v4.3.rc2 - Arch Linux #1 SMP PREEMPT_DYNAMIC Sat, 27 Jul 2024 16:49:55 +0000 - Wayland - GLES3 (Compatibility) - AMD Radeon RX 6600 (radeonsi, navi23, LLVM 18.1.8, DRM 3.57, 6.10.2-arch1-1) - AMD Ryzen 7 5700G with Radeon Graphics (16 Threads)

Issue description

The project name reverts back to "New Game Project" after browsing for a project path. Also the text is highlighted after closing the browse window but nothing on the "Create New Project" window is focused. It happens on linux with the editor set to either x11 or wayland.

Steps to reproduce

  • Press "Create" button in project manager
  • Type new project name
  • Press "Browse" button

Minimal reproduction project (MRP)

N/A

@akien-mga akien-mga added this to the 4.3 milestone Aug 1, 2024
@akien-mga akien-mga moved this from Unassessed to Not Critical in 4.x Release Blockers Aug 1, 2024
@akien-mga akien-mga moved this from Not Critical to Bad in 4.x Release Blockers Aug 1, 2024
@akien-mga
Copy link
Member

Seems to be a regression from #94113, CC @jsjtxietian @KoBeWi.

I also see this error in the console when following the steps to reproduce:

ERROR: Attempting to make child window exclusive, but the parent window already has another exclusive child. This window: /root/@ProjectManager@1111/@EditorFileDialog@1253, parent window: /root, current exclusive child window: /root/@ProjectManager@1111/@ProjectDialog@697
   at: _set_transient_exclusive_child (./scene/main/window.cpp:949)

Reverting #94113 solves the error and the reported regression.

@jsjtxietian
Copy link
Contributor

jsjtxietian commented Aug 2, 2024

The project name reverts back to "New Game Project" after browsing for a project path.

This is because the show_dialog is called for every selected project path and it will reset the name no matter what, should be an easy fix.

Attempting to make child window exclusive, but the parent window already has another exclusive child. This window: /root/@projectmanager@1111/@EditorFileDialog@1253, parent window: /root, current exclusive child window: /root/@projectmanager@1111/@ProjectDialog@697

Looks like we can't pop up two seperate window at a time, so maybe it's not a good idea to use add_sibling ? Or hide self first before poping.

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

Successfully merging a pull request may close this issue.

3 participants