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

Fix ProjectManager import dialog error #100197

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

molingyu
Copy link

@molingyu molingyu commented Dec 9, 2024

When Cancel is selected in the import dialog, no subsequent operations should be performed.

The signal in the original code was incorrectly connected to canceled instead of confirmed. This results in that after canceling the import dialog, subsequent operations are still performed and the default path is used as the path for the imported project.

7a5c3111fcfa7c1c7700bd8385848366

When Cancel is selected in the import dialog, no subsequent operations should be performed.
@molingyu molingyu requested a review from a team as a code owner December 9, 2024 10:16
@AThousandShips AThousandShips added this to the 4.4 milestone Dec 9, 2024
@fire fire changed the title fix ProjectManager import dialog error Fix ProjectManager import dialog error Dec 9, 2024
@AThousandShips AThousandShips requested a review from KoBeWi December 9, 2024 10:28
@akien-mga
Copy link
Member

This code was introduced in #95062, so this would be a regression / bug introduced in 4.3.

@akien-mga akien-mga added regression cherrypick:4.3 Considered for cherry-picking into a future 4.3.x release labels Dec 9, 2024
@KoBeWi
Copy link
Member

KoBeWi commented Dec 27, 2024

Canceling should go back to Create New Project dialog, now it closes it completly.

godot.windows.editor.dev.x86_64_fyFDn8BofZ.mp4

vs

godot_Ly3jxXOmwG.mp4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug cherrypick:4.3 Considered for cherry-picking into a future 4.3.x release regression topic:editor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants