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

Check UISwitch state using combination of trait and value #170

Merged
merged 1 commit into from
Nov 17, 2023

Conversation

NickEntin
Copy link
Collaborator

No description provided.

@NickEntin NickEntin force-pushed the entin/switch-state-by-value branch from 46b7627 to b0d729d Compare November 10, 2023 23:50
@NickEntin

This comment was marked as resolved.

@NickEntin
Copy link
Collaborator Author

Check what happens when the accessibilityValue is a non-0 or 1 numeric value

From what I can tell there are three special cased values:

  • 0 => "off"
  • 1 => "on"
  • 2 => "mixed"

I'm not sure what the last one is used for, but simple enough to support with the same pattern.

@NickEntin
Copy link
Collaborator Author

Check if the outputted values are localized by numeric script

Nope, looks like it's always 0 and 1

@NickEntin NickEntin force-pushed the entin/switch-state-by-value branch from b0d729d to 9614f39 Compare November 15, 2023 03:31
@NickEntin

This comment was marked as resolved.

@NickEntin NickEntin force-pushed the entin/switch-state-by-value branch from 9614f39 to b2c018d Compare November 15, 2023 07:28
@NickEntin
Copy link
Collaborator Author

Check what happens when the accessibilityValue is a non-0 or 1 numeric value

From what I can tell there are three special cased values:

  • 0 => "off"
  • 1 => "on"
  • 2 => "mixed"

I'm not sure what the last one is used for, but simple enough to support with the same pattern.

Oh, VoiceOver probably shares logic with macOS, which has tri-state controls (e.g. checkboxes).

@NickEntin NickEntin force-pushed the entin/switch-state-by-value branch from b2c018d to 76cfb1e Compare November 16, 2023 05:19
@NickEntin NickEntin marked this pull request as ready for review November 16, 2023 05:21
@NickEntin NickEntin requested a review from meherkasam November 16, 2023 05:21
@NickEntin NickEntin force-pushed the entin/switch-state-by-value branch from 76cfb1e to 9a0db5a Compare November 16, 2023 05:27
@NickEntin NickEntin merged commit 9a72e64 into master Nov 17, 2023
6 checks passed
@NickEntin NickEntin deleted the entin/switch-state-by-value branch November 17, 2023 01:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants