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

[DOCS] added several wayland tips #4795

Closed
wants to merge 10 commits into from

Conversation

holykremowka2137
Copy link
Contributor

Me and @wingej0 wrote several tips, so more people can switch to this backend

@tych0
Copy link
Member

tych0 commented Apr 30, 2024

Docs themselves look great, thank you for that. Can you clean up the git history in the branch?

@elParaguayo
Copy link
Member

Some of these edits aren't deleting the old text, they're just added after them so we get repeated sections.

I can highlight them later, difficult to do on my phone.

@wingej0
Copy link
Contributor

wingej0 commented Apr 30, 2024

Some of these edits aren't deleting the old text, they're just added after them so we get repeated sections.

I can highlight them later, difficult to do on my phone.

I fixed the document. I sent it back to @holykremowka2137. They will have to push the changes to the repo.

And add this to your autostart script:

.. code-block:: sh
dbus-update-activation-environment --systemd WAYLAND_DISPLAY XDG_CURRENT_DESKTOP &
Copy link
Contributor

@jwijenbergh jwijenbergh Apr 30, 2024

Choose a reason for hiding this comment

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

this is not guaranteed to work as you might also have to restart your systemd services to take effect, see step 5 here https://github.com/emersion/xdg-desktop-portal-wlr/wiki/%22It-doesn't-work%22-Troubleshooting-Checklist

import os
from libqtile import qtile
if qtile.core.name == "wayland":
os.environ["XDG_SESSION_DESKTOP"] = "qtile:wlroots"
Copy link
Contributor

Choose a reason for hiding this comment

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

afaik XDG_SESSION_DESKTOP should be unneeded, and is also not standardized


Here is a write up on how to get screen sharing enabled in Wayland:

To get screen sharing working on Qtile Wayland, make sure `xdg-desktop-portal`, `xdg-desktop-portal-gtk`, `xdg-desktop-portal-wlr`, `python-dbus-next`, and `wlroots` are installed on your system and add the following lines to your `config.py`:
Copy link
Contributor

Choose a reason for hiding this comment

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

dbus-next python dep is not needed, only qtile to be ran with dbus-run-session. Remove wlroots here as it's always needed

@@ -51,7 +53,80 @@ Input Device Configuration
If you want to change keyboard configuration during runtime, you can use the
core's `set_keymap` command (see below).

*It might be good to add something here about what works and what doesn't. For instance `"type:keyboard": InputConfig(kb_numlock=True)` doesn't work. In fact, it makes it so my numlock won't stay on.*
Copy link
Contributor

Choose a reason for hiding this comment

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

remove this

@@ -38,6 +38,8 @@ Qtile supports XWayland but requires that `wlroots` and `pywlroots` were built
with XWayland support, and that XWayland is installed on the system from
startup. XWayland will be started the first time it is needed.

Edit: In order to have XWayland support in Qtile, `wlroots` and `pywlroots` must be built with XWayland support, and XWayland must be installed on the system at startup. It will be started the first time it is needed.
Copy link
Contributor

Choose a reason for hiding this comment

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

qtile does not have optional xwayland support afaik, you have to build with it

Comment on lines +81 to +83
GTK have a known issue of not recognizing setted variables. You can fix this by saving below code into a script in your script directory. Then you can just put the name of the script into your autostarting script.

Edit: GTK has a known issue of not recognizing set variables. To fix this, create a script with the code below and save it in your script directory. Then add the script to your autostart script. For a GUI solution, the program `nwg-look` works well.
Copy link
Contributor

Choose a reason for hiding this comment

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

duplicate sentences here

@jwijenbergh
Copy link
Contributor

Thanks for this! I added some comments. FWIW I am currently writing a wayland guide, but some of the tips from this are not in there yet. If my nits are fixed in the review then this can be merged and I will adapt my guide to include all the tips from here too

Copy link

This PR is stale because it has been open 90 days with no activity. Remove the status: stale label or comment, or this will be closed in 30 days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants