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

Some command palette commands do nothing #78981

Closed
Kregap opened this issue Jul 3, 2023 · 6 comments · Fixed by #82203
Closed

Some command palette commands do nothing #78981

Kregap opened this issue Jul 3, 2023 · 6 comments · Fixed by #82203

Comments

@Kregap
Copy link

Kregap commented Jul 3, 2023

Godot version

v4.1.rc2.official [4642448]

System information

Windows 11

Issue description

Selecting some of the commands from command palette do nothing. Seems like commands that should open a popup/window fail, while other commands work. Selecting faulty commands from menu directly work.

cmd-palette.mp4

Steps to reproduce

  1. Open commands palette (Ctrl+Shift+P).
  2. Select "Editor Settings" from the list.
  3. Confirm command selection with either return keyboard key or by clicking on "OK" button.
    Expected behaviour
    Command palette popup hides.
    Editor Settings popup opens.
    Actual behaviour
    Command palette popup hides but Editor Settings popup does not open.

Minimal reproduction project

N/A

@Sauermann
Copy link
Contributor

I was able to bisect this to #76719, which was also ported to 3.x.

@akien-mga akien-mga added this to the 4.2 milestone Jul 3, 2023
@aryyya
Copy link

aryyya commented Jul 10, 2023

This also happens for me on macOS Ventura and v4.1.stable.official.

Many command palette commands do nothing. Additionally the hotkeys assigned to certain actions also do nothing.

For example both the command palette and hotkey for "toggle distraction free mode" do nothing.

@passivestar
Copy link
Contributor

Same problem.
Godot v4.1.1.stable - macOS 13.5.2

Trying to execute "Project Settings", "Export", "New Scene", etc has no effect

@SirLich
Copy link
Contributor

SirLich commented Sep 22, 2023

Just a heads-up that I am actively trying to solve this bug. I think I've got some good guesses, but until I can attach a debugger it will be tricky.

I'm quite suspicious of this snippet:

void EditorCommandPalette::_confirmed() {
	TreeItem *selected_option = search_options->get_selected();
	String command_key = selected_option != nullptr ? selected_option->get_metadata(0) : "";
	if (!command_key.is_empty()) {
		hide();
		execute_command(command_key);
	}
}

@mcdz
Copy link

mcdz commented Sep 28, 2023

Same issue in v4.2 dev 5

@kleonc
Copy link
Member

kleonc commented Sep 28, 2023

Same issue in v4.2 dev 5

@mcdz v4.2.dev5 was released 9 days ago, #82203 (fixing this issue) was merged 4 days ago and hence is not included in 4.2.dev5. It will be included in the next version.

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.

10 participants