Skip to content

[PS-672] Accessibility - File/Text controls unintuitive, "Text" accessible name incorrect#1923

Merged
andrebispo5 merged 5 commits intomasterfrom
bug/ps-672-send-voiceover
May 25, 2022
Merged

[PS-672] Accessibility - File/Text controls unintuitive, "Text" accessible name incorrect#1923
andrebispo5 merged 5 commits intomasterfrom
bug/ps-672-send-voiceover

Conversation

@andrebispo5
Copy link
Contributor

Type of change

  • Bug fix
  • New feature development
  • Tech debt (refactoring, code cleanup, dependency upgrades, etc)
  • Build/deploy pipeline (DevOps)
  • Other

Objective

Change the segmented button (File/Text) in Sends tab, to have a better voice feedback to the user using TalkBack or VoiceOver. Added a missing "Text" resource and dynamic voice over based on button state.

Code changes

  • src/App/Pages/Send/SendAddEditPageViewModel.cs: Label properties for button states
  • src/App/Pages/Send/SendAddEditPage.xaml.cs: Accessibility help text change on button click and screen init.

Before you submit

  • I have added unit tests where it makes sense to do so (encouraged but not required)
  • This change requires a documentation update (notify the documentation team)
  • This change has particular deployment requirements (notify the DevOps team)

…ible name incorrect

- Added new text to help identify File / Text segmented button
- Added missing text for Text button accessibility
@andrebispo5 andrebispo5 requested a review from fedemkr May 24, 2022 21:33
@andrebispo5 andrebispo5 requested a review from fedemkr May 25, 2022 10:15
@andrebispo5 andrebispo5 merged commit 43e9515 into master May 25, 2022
@andrebispo5 andrebispo5 deleted the bug/ps-672-send-voiceover branch May 25, 2022 16:20
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.

Android/iOS: Accessibility - File/Text controls unintuitive, "Text" accessible name incorrect

3 participants