-
-
Notifications
You must be signed in to change notification settings - Fork 696
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
Conversation
Docs themselves look great, thank you for that. Can you clean up the git history in the branch? |
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 & |
There was a problem hiding this comment.
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" |
There was a problem hiding this comment.
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`: |
There was a problem hiding this comment.
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.* |
There was a problem hiding this comment.
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. |
There was a problem hiding this comment.
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
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. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
duplicate sentences here
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 |
This PR is stale because it has been open 90 days with no activity. Remove the |
Me and @wingej0 wrote several tips, so more people can switch to this backend