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

Introduce WindowActorFetcher #2176

Closed
wants to merge 3 commits into from
Closed

Conversation

lenemter
Copy link
Member

@lenemter lenemter commented Dec 19, 2024

I want to use this as a replacement for Meta.Window.shown since it has proven to be wrong in e.g. #2088.

I don't even think Meta.Window.shown was intended to be used for determining whether the window actor was shown. But in short, when the new window is created the Wayland windows are 'hidden' by default but X11 are not. because of this the Meta.Window.shown sometimes is not called.

The code here only touches WindowClone to show that it works, I'll propose PR to replace the use of 'shown' later.

@lenemter lenemter requested a review from a team December 19, 2024 10:09
@lenemter lenemter closed this Dec 19, 2024
@lenemter lenemter deleted the lenemter/introduce-window-fetcher branch December 19, 2024 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant