jamulus icon indicating copy to clipboard operation
jamulus copied to clipboard

Try to fix QTreeWidget accessibility bug

Open ann0see opened this issue 1 month ago • 1 comments

Co-authored-by: GitHub Copilot

This might contain a lot of low quality code for now.

How it looks like:

image

There is a "Show/Hide accessible controls button" at the top which shows/hides the previous/next buttons + the current server info label.

Short description of changes Adds forward/backward buttons and label to connect dialog for accessability Copilot's suggestion to fix: #3559 (together with some tweaking)

CHANGELOG: Accessability: Add buttons and label to go through server list

Context: Fixes an issue? Yes: #490 (workaround)

Does this change need documentation? What needs to be documented and how?

Yes

Status of this Pull Request

Proof of concept. The code is probably extremely convoluted.

What is missing until this pull request can be merged?

Testing, review and simplification.

Checklist

  • [ ] I've verified that this Pull Request follows the general code principles
  • [x] I tested my code and it does what I want
  • [ ] My code follows the style guide
  • [ ] I waited some time after this Pull Request was opened and all GitHub checks completed without errors.
  • [ ] I've filled all the content above

AUTOBUILD: Please build all targets

ann0see avatar Dec 06 '25 12:12 ann0see

@chigkim this PR doesn't need the additional flag at the moment. Please tab through the UI until you get to the button/textfield and use the buttons to go forward/backward and use the textbox to hear the server info. It's not the most usable at the moment but at least it should work to some extent.

I tested it on macOS 26.1 with voiceover and it did work

ann0see avatar Dec 11 '25 09:12 ann0see

In the best case, we'd extract a row and let VoiceOver read the server name. Then with a button press, one would move forward/backward etc. I'll give this method some more thought.

ann0see avatar Dec 19 '25 21:12 ann0see