-
-
Notifications
You must be signed in to change notification settings - Fork 28
Flatpak: add accessibility overrides #343
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
base: main
Are you sure you want to change the base?
Conversation
| @@ -0,0 +1,6 @@ | |||
| [Environment] | |||
| QT_LINUX_ACCESSIBILITY_ALWAYS=true | |||
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 should have been QT_LINUX_ACCESSIBILITY_ALWAYS_ON=1 but it is not required as the documentation stands:
In the Unix/X11 AT-SPI implementation, applications become accessible when two conditions are met:
org.a11y.Status.IsEnabled DBus property is true
org.a11y.Status.ScreenReaderEnabled DBus property is trueAn alternative to setting the DBus AT-SPI properties is to set the QT_LINUX_ACCESSIBILITY_ALWAYS_ON environment variable.
So the org.a11y.bus=talk should be enough
| @@ -0,0 +1,6 @@ | |||
| [Environment] | |||
| QT_LINUX_ACCESSIBILITY_ALWAYS=true | |||
| GTK_PATH=/app/lib/gtkmodules | |||
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 shouldn't be required, the AT-SPI module is built with Gtk itself so the default paths should be enough
|
I don't see anybody else doing this, I suppose that they still have accessibility support though so this sounds like we are doing something wrong elsewhere… |
Actually, I didn't even know about this, I imagine basically nobody else did either. I need to figure out how to ship something like this for Fedora KDE. |
|
Tracking this in Fedora KDE: https://pagure.io/fedora-kde/SIG/issue/640 |
Fixes #342
This will only apply to new users, so... we probably should create this file manually in packaging I guess. Also it doesn't apply to system applications so I need to figure out how to write this file in /var/lib/flatpak/overrides. If you know how to do this feel free to push into this branch.