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

Don't process input in hidden EditorProperty. #63197

Merged
merged 1 commit into from
Aug 8, 2022

Conversation

rcorre
Copy link
Contributor

@rcorre rcorre commented Jul 19, 2022

This causes EditorProperty nodes to intercept input events even when the
Editor Properties dialog is not visible. This means that after closing
the dialog, ctrl+shift+c will still copy the last selected property
path.

Fixes #62866.

4.0 uses shortcut_input instead of unhandled_input, which does not have the same issue.
shortcut_input is not available in 3.x, so we have to work around it.

This causes EditorProperty nodes to intercept input events even when the
Editor Properties dialog is not visible. This means that after closing
the dialog, ctrl+shift+c will still copy the last selected property
path.

Fixes godotengine#62866.
@YeldhamDev YeldhamDev added this to the 3.x milestone Jul 19, 2022
@YeldhamDev YeldhamDev linked an issue Jul 19, 2022 that may be closed by this pull request
@akien-mga akien-mga requested review from KoBeWi and YuriSizov August 8, 2022 13:43
Copy link
Member

@KoBeWi KoBeWi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks ok. AFAIK we have similar code in other places (e.g. #58251 happened when visibility was broken).

@akien-mga akien-mga merged commit c18ef0d into godotengine:3.x Aug 8, 2022
@akien-mga
Copy link
Member

Thanks!

@akien-mga akien-mga added the cherrypick:3.5 Considered for cherry-picking into a future 3.5.x release label Aug 8, 2022
@akien-mga akien-mga modified the milestones: 3.x, 3.6 Aug 8, 2022
@rcorre rcorre deleted the fix-copy-node-path branch August 9, 2022 10:45
@akien-mga
Copy link
Member

Cherry-picked for 3.5.1.

@akien-mga akien-mga removed the cherrypick:3.5 Considered for cherry-picking into a future 3.5.x release label Aug 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't copy node path with Ctrl-Shift-C if Property selected.
4 participants