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

PanelWindow: divide dimensions by scale on Wayland #563

Closed
wants to merge 2 commits into from

Conversation

danirabbit
Copy link
Member

@danirabbit danirabbit commented Aug 15, 2024

Fixes #562

Test panel size on:

  • HiDPI with Wayland
  • LoDPI with Wayland
  • HiDPI with X11
  • LoDPI with X11

@danirabbit
Copy link
Member Author

danirabbit commented Aug 15, 2024

Can't fully test opening indicators on X11 because of #564 :( But theoretically this only does things on Wayland so 🤷‍♀️

@danirabbit danirabbit requested a review from a team August 15, 2024 22:05
@danirabbit danirabbit marked this pull request as ready for review August 15, 2024 22:05
@leolost2605
Copy link
Member

It seems this is only needed when fractional scaling is disabled (since it does logical layout instead of physical). Otherwise it will reintroduce the issue fixed in #559. Maybe we can do it depending on whether it's enabled?

@danirabbit
Copy link
Member Author

@leolost2605 ah is there a simple way to test for that?

@leolost2605
Copy link
Member

Hmm I dug a bit through gtk api but I only found stuff for GTK4 so I was kinda hoping somebody else knew something. I'll prepare a branch where we ask mutter via dbus :)

@danirabbit
Copy link
Member Author

Closing since you proposed a better branch

@danirabbit danirabbit closed this Aug 20, 2024
@danirabbit danirabbit deleted the danirabbit/panelwindow-waylanddimensions branch August 20, 2024 20:14
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.

Most indicators missing on Wayland
2 participants