Skip to content

interfaces: Add orca-screen-reader interface - #17397

Draft
sergio-costas wants to merge 6 commits into
canonical:masterfrom
sergio-costas:add-snapped-orca-support
Draft

interfaces: Add orca-screen-reader interface#17397
sergio-costas wants to merge 6 commits into
canonical:masterfrom
sergio-costas:add-snapped-orca-support

Conversation

@sergio-costas

@sergio-costas sergio-costas commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

To support a snapped Orca screen reader, a new interface is required to give to it full access to the accessibility bus.

This patch adds a specific interface that allows the Orca screen reader to have full access to the required elements, and also allows any snapped program to send accessibility events to the screen reader.

To simplify the design and guarantee backwards compatibility, the snap name for the screen reader is hardcoded. This results in a much simpler and straightforward patch than #16292 , but less flexible (but since there aren't in the horizon any replacement for Orca, that shouldn't be a problem).

@github-actions github-actions Bot added the Run only one system Only runs spread tests on one system label Jul 23, 2026
@github-actions

github-actions Bot commented Jul 23, 2026

Copy link
Copy Markdown

Thu Jul 23 13:14:33 UTC 2026
The following results are from: https://github.com/canonical/snapd/actions/runs/30007657818

No spread failures reported

Skipped tests from snapd-testing-skip

If you wish to have any of the below tests run in your PR, in your PR description, add 'unskip:' followed by a copy-and-pasted list of the below tests you wish to run (unskip plus test list must be valid yaml)

  • openstack:ubuntu-24.04-64:tests/main/i18n

@codecov

codecov Bot commented Jul 23, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 84.21053% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.87%. Comparing base (bc3f673) to head (c4831f3).
⚠️ Report is 8 commits behind head on master.

Files with missing lines Patch % Lines
interfaces/builtin/orca_screen_reader.go 81.25% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master   #17397      +/-   ##
==========================================
- Coverage   78.88%   78.87%   -0.02%     
==========================================
  Files        1399     1406       +7     
  Lines      196140   196337     +197     
  Branches     2462     2462              
==========================================
+ Hits       154718   154851     +133     
- Misses      32136    32200      +64     
  Partials     9286     9286              
Flag Coverage Δ
unittests 78.87% <84.21%> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

To support a snapped Orca screen reader, a new interface is
required to give to it full access to the accessibility bus.

This patch adds a specific interface that allows the Orca screen
reader to have full access to the required elements, and also
allows any snapped program to send accessibility events to the
screen reader.

To simplify the design and guarantee backwards compatibility,
the snap name for the screen reader is hardcoded.
@sergio-costas
sergio-costas force-pushed the add-snapped-orca-support branch from 74b24ce to c4831f3 Compare July 23, 2026 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Run only one system Only runs spread tests on one system

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant