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

Enable Wayland display server #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MartinBriza
Copy link

This PR makes the package able to connect to the Wayland display server in addition to Xorg. The biggest effect this has is that the application won't be blurry (or small) with high DPI scaling.

@flathubbot
Copy link
Contributor

Started test build 55605

@nuttyartist
Copy link
Collaborator

Pinging @guihkx for feedback.

@flathubbot
Copy link
Contributor

Build 55605 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/38262/io.github.nuttyartist.notes.flatpakref

@guihkx
Copy link
Collaborator

guihkx commented Jul 28, 2023

Wayland support was disabled on purpose by me because I honestly found the overall experience a bit inferior to running the app through XWayland.

But I hadn't experienced (or even thought about) HiDPI issues, so that might a good argument to enable it.

But before we merge this, can you please remove the now redundant --socket=x11 permission first? Thanks!

@guihkx
Copy link
Collaborator

guihkx commented Jul 28, 2023

@zjeffer just reported this new bug:

Then perhaps we should postpone this change? Especially considering that the Kanban view is a paid feature, so we can't afford breaking it, in my opinion.

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.

5 participants