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

GNOME 47 Scaling #33

Merged
merged 3 commits into from
Sep 26, 2024
Merged

GNOME 47 Scaling #33

merged 3 commits into from
Sep 26, 2024

Conversation

fossfreedom
Copy link
Contributor

This is a truncated version of https://gitlab.gnome.org/GNOME/mutter/-/merge_requests/3567/diffs that ensures gnome-settings-daemon xsettings has something to chew on.

It creates a dbus interface for GSD to obtain scaling info.

magpie itself responds to monitors signal changes to configure the UI scaling factor.

I've given this a preliminary look-see in a virtualbox session scaling between 100%, 200% and 300% and the scaling survives logout and login.

Well worth retesting with multimonitors etc.

This code  creates the dbus interface allowing GSD to call
the dbus API.
There is now a new UIScalingFactor property.
Next is to work out how BCC signals that a scaling change
has occurred.
@fossfreedom fossfreedom requested a review from a team as a code owner September 23, 2024 19:00
Copy link
Member

@EbonJaeger EbonJaeger left a comment

Choose a reason for hiding this comment

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

I can't test this as we don't have GNOME 47 on Solus, but the code itself looks fine to me.

@fossfreedom
Copy link
Contributor Author

@EbonJaeger thx! I've patched UB 24.10 and team-member Jacob has confirmed 150% fractional scaling all works. So happy from this side. Would like another distro to confirm as well before merging.

@JoshStrobl JoshStrobl self-requested a review September 26, 2024 08:12
Copy link
Member

@JoshStrobl JoshStrobl left a comment

Choose a reason for hiding this comment

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

Added a squashed patch of this to the Fedora magpie package, cooked locally and tested in a Fedora Budgie 41 nightly image. Can confirm that scaling works as expected. Fired up another app (Nemo) and it has the correct scaling as well.

Screenshot from 2024-09-26 11-50-31

@fossfreedom fossfreedom merged commit 263b41a into main Sep 26, 2024
1 check passed
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.

3 participants