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

X11: move dock window out of bounds when hidden #2122

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

lenemter
Copy link
Member

@lenemter lenemter commented Nov 23, 2024

Fixes #2101

This is very very bad code but I just couldn't find other solution. So far I tried:

  • Minimizing window
  • Using _NET_WM_STATE_HIDDEN (which is basically minimizing AFAIK)
  • Blocking X11 window events by changing its event mask
  • Using XShape extension

Nothing worked. If you know any other solution please let me know.

@lenemter lenemter requested a review from a team November 23, 2024 15:44
@leolost2605
Copy link
Member

leolost2605 commented Nov 23, 2024

Tbh I think at some point we should just stop doing workarounds and instead go back to shipping plank for X since the whole infrastructure here (i.e. the new dock, the protocol, everything in /ShellClients) was never intended, tested or designed to work with X11 and just retrofitted into it with hopes it will work.
And IMHO this whole thing just gets messier and messier and therefore hard to maintain since always two scenarios have to be tested all at the risk of reducing quality for all sessions.
I'm not saying this point should be now (although I'd like it to be) but we should at least think about it.

@lenemter lenemter mentioned this pull request Nov 23, 2024
1 task
@lenemter
Copy link
Member Author

@leolost2605 I don't agree with that. If we don't adapt gala's code to X11, the same bugs will happen with Wingpanel when we'll implement elementary/wingpanel#53 for example. And suddenly we need to ship two versions of the same component

@leolost2605
Copy link
Member

leolost2605 commented Nov 24, 2024

Something like elementary/wingpanel#53 would be pretty much the only reason to support this on X and I think it's not worth polluting the codebase for that. Since there also doesn't seem to be a lot of movement regarding it I don't see it as a valid argument right now.
And even if something like elementary/wingpanel#53 would be agreed upon I'd argue that it can land in Wayland only because X11 can be considered deprecated and can hopefully be completely dropped very soon.
I don't think it's worth supporting it at the cost of a messy codebase especially since afaik apart from very specific edge cases Wayland provides a better experience as of today (tbh I also don't quite get why X is still the default for OS8).

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.

Dock interacts with the mouse even with the window overlapping
2 participants