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

[BUG] Range needs VoiceOver support for alternating values #3776

Open
1 of 9 tasks
chi-jyskebank-dk opened this issue Jan 28, 2025 · 0 comments
Open
1 of 9 tasks

[BUG] Range needs VoiceOver support for alternating values #3776

chi-jyskebank-dk opened this issue Jan 28, 2025 · 0 comments
Labels
accessibility bug 👶🏻 New For new issues before prioritisation and refinement NOT Prioritized Issue not yet prioritized and added to a Milestone NOT Verified Applied to bug reports that have not been verified by a member of @kirbydesign/kirby-guild

Comments

@chi-jyskebank-dk
Copy link
Collaborator

Describe the bug

VoiceOver is only able to read out the current value of a Range slider. It would make sense to be able to programatically specify the text that is read out, eg. the value + a unit, or perhaps more advanced requirements for alternating texts.

Describe how to reproduce the bug

  1. Go to StackBlitz: https://stackblitz.com/edit/stackblitz-starters-wxb9gweu?file=src%2Fmain.ts
  2. Ensure kirby-range [pin]='false'
  3. When adjusting the Range slider, only the new value is read out loud

Which Kirby version was used?

10.2.1

What was the expected behavior?

It would be much preferred if the component consuming the Range, could expose a function with logic that could report to the Range what was supposed to be read out by VoiceOver. Eg. "100.000 kr.", "30 shares" or "55.000 kr., higher than recommended".

Add any screenshots

Sample screen recording, showing how the current [pinFormatter] is used to simulate the expected behaviour:
https://github.com/user-attachments/assets/254e0ad5-09b5-44ec-b6f4-213f1085e1d7

Please complete the following information:

  • OS: macOS 15.2 with VoiceOver
  • Browser: Google Chrome

Are there any additional context?

I suggest a new optional callback is offered by the Range component, eg. [ariaSliderAdjustCallback]="someFunction", which will allow the consuming component, to decide the text read out by VoiceOver, based on the current value of the slider. Very much like the [pinFormatter] works.

In the StackBlitz mentioned above, set kirby-range [pin]='true', to get an idea of what kind of flexibility we wish for when using VoiceOver (think of the values shown in the Pin, as values that would instead be read out to the user). In this case it doesn't make sense to show the text in a pin, as it is rather long, but make perfect sense when being read out loud.


Checklist:

The following tasks should be carried out in sequence in order to follow the process of contributing correctly.

Verification

To make sure the bug is not intended behaviour; it should be verified by a member of team Kirby before moving on to implementation.

  • Make sure the NOT verified label has been removed by a member of team Kirby; do not proceed until this is done.

Implementation

The contributor who wants to implement this issue should:

Review

Once the issue has been implemented and is ready for review:

@chi-jyskebank-dk chi-jyskebank-dk added accessibility bug NOT Prioritized Issue not yet prioritized and added to a Milestone NOT Verified Applied to bug reports that have not been verified by a member of @kirbydesign/kirby-guild 👶🏻 New For new issues before prioritisation and refinement labels Jan 28, 2025
@github-project-automation github-project-automation bot moved this to 💌 Inbox in Kirby Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility bug 👶🏻 New For new issues before prioritisation and refinement NOT Prioritized Issue not yet prioritized and added to a Milestone NOT Verified Applied to bug reports that have not been verified by a member of @kirbydesign/kirby-guild
Projects
Status: 💌 Inbox
Development

No branches or pull requests

1 participant