-
Notifications
You must be signed in to change notification settings - Fork 10
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
feat(281249): improve Login and Create Account page reader support #313
base: feature/screen-reader-support
Are you sure you want to change the base?
feat(281249): improve Login and Create Account page reader support #313
Conversation
src/app/ApplicationTemplate.Shared.Views/Styles/Controls/CheckBox.xaml
Outdated
Show resolved
Hide resolved
...app/ApplicationTemplate.Presentation/ViewModels/Authentication/CreateAccountPageViewModel.cs
Outdated
Show resolved
Hide resolved
src/app/ApplicationTemplate.Shared.Views/Styles/Controls/DataValidationView.xaml
Outdated
Show resolved
Hide resolved
@Arieldelossantos merge latest main to fix your pipeline |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, just need to fix comments. So if I understand the goal of this Pull Request is to make the choices compatible with a reader?
src/app/ApplicationTemplate.Shared.Views/Content/Authentication/CreateAccountPage.xaml
Outdated
Show resolved
Hide resolved
src/app/ApplicationTemplate.Shared.Views/Content/Authentication/CreateAccountPage.xaml
Outdated
Show resolved
Hide resolved
src/app/ApplicationTemplate.Shared.Views/Content/Authentication/CreateAccountPage.xaml
Outdated
Show resolved
Hide resolved
with the accessibility voiceover |
8149db7
to
cf1119b
Compare
...icationTemplate.Shared.Views/Converters/FromDataValidationStateToVoiceOverStringConverter.cs
Outdated
Show resolved
Hide resolved
...icationTemplate.Shared.Views/Converters/FromDataValidationStateToVoiceOverStringConverter.cs
Outdated
Show resolved
Hide resolved
cf1119b
to
7938e50
Compare
7938e50
to
f277f99
Compare
PR Type
What kind of change does this PR introduce?
Description
improve Login and Create Account page reader support
PR Checklist
Please check if your PR fulfills the following requirements:
Other information
Product Backlog Item 281249: Improve screen reader support in UnoAppTemplate
Internal Issue (If applicable):