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

Implement a transient_to_focused Window mode #86446

Merged
merged 1 commit into from
Jan 15, 2024

Conversation

reduz
Copy link
Member

@reduz reduz commented Dec 22, 2023

This intends to be the correct way to handle non-child windows becoming covered by the current window when becoming focused. Enabling this property on select windows, they will become transient to the currently focused one when becoming visible.

This deprecates the "unparent_when_invisible" function introduced by #76025.

@reduz reduz requested review from a team as code owners December 22, 2023 16:54
@AThousandShips AThousandShips added this to the 4.x milestone Dec 26, 2023
@bruvzg bruvzg self-requested a review December 26, 2023 20:24
@akien-mga akien-mga changed the title Implement a transient_to_focused mode Implement a transient_to_focused mode Jan 8, 2024
@akien-mga akien-mga changed the title Implement a transient_to_focused mode Implement a transient_to_focused Window mode Jan 8, 2024
@akien-mga akien-mga requested review from a team and removed request for a team January 8, 2024 13:18
@YuriSizov YuriSizov self-requested a review January 8, 2024 15:28
@YuriSizov YuriSizov modified the milestones: 4.x, 4.3 Jan 8, 2024
scene/main/window.cpp Outdated Show resolved Hide resolved
scene/main/window.cpp Outdated Show resolved Hide resolved
Copy link
Contributor

@YuriSizov YuriSizov left a comment

Choose a reason for hiding this comment

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

Left a couple of comments based on code review, which need to be addressed. But in testing (rebased on master and with clear_transient comment fixed) everything seems to be in working order. Changes are clear and make sense.

scene/main/window.cpp Outdated Show resolved Hide resolved
scene/main/window.cpp Outdated Show resolved Hide resolved
doc/classes/Window.xml Outdated Show resolved Hide resolved
This intends to be the correct way to handle non-child windows becoming covered by the current window when becoming focused.
Enabling this property on select windows, they will become transient to the currently focused one when becoming visible.

This deprecates the "unparent_when_invisible" function introduced by godotengine#76025.
@reduz reduz force-pushed the transient-to-focused branch from f99be9d to 15144c2 Compare January 14, 2024 17:52
@reduz
Copy link
Member Author

reduz commented Jan 14, 2024

Okay, implemented suggested changes.

@akien-mga akien-mga merged commit 03767fb into godotengine:master Jan 15, 2024
15 checks passed
@akien-mga
Copy link
Member

Thanks!

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.

6 participants