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

The FileDialog > Create New Folder popup should also be exclusive and not hide if you click on it's parent FileDialog pane. #92848

Open
wyattbiker opened this issue Jun 6, 2024 · 1 comment · May be fixed by #94488

Comments

@wyattbiker
Copy link

wyattbiker commented Jun 6, 2024

Tested versions

System information

Linux Kubuntu 23.04 KDE Plasma Version: 5.27.4 KDE Frameworks Version: 5.104.0 Qt Version: 5.15.8

Issue description

When using the FileDialog node, the Create Folder popup should be forced to exclusive. If you click away by accident (e.g. inside the the FileDialog panel), the popup hides behind the FileDialog panel and cannot be recovered (unless you know to drag the FileDialog away to find it). In that case you need to force close the application.

image

Steps to reproduce

See reproduction attachment below.

  • Create a main Node scene and add a FileDialog node with visible, exclusive, centered, size of main window (I used a small code snippet to position and size).
  • Run the project
  • Click on Create new folder button to the top right of the FileDialog
  • The Create Folder popup should appear.
  • Click inside the FileDialog pane
  • The Create Folder popup hides behind. Cannot be recovered (unless you were able to drag the File Dialog pane away)

Minimal reproduction project (MRP)

Archive.zip

@wyattbiker
Copy link
Author

wyattbiker commented Jun 11, 2024

I wrote this function for now to fix the problem. It's specific to my issue but works. Can be modified to fix AcceptDialog as well.

fix_confirmations($SomeFD)	

func fix_confirmations(this: FileDialog):
	var node_list=this.get_children(true)	
	for node in node_list:
		if node.is_class("ConfirmationDialog"):
			node.always_on_top=true

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

Successfully merging a pull request may close this issue.

2 participants