felixfoertsch/bundesmessenger-ios
1
0
Fork 0
mirror of https://gitlab.opencode.de/bwi/bundesmessenger/clients/bundesmessenger-ios.git synced 2026-04-18 07:28:28 +02:00
Code Issues Packages Projects Releases Wiki Activity

Add accessibility labels/hints. Fix tests. Show analytics prompt to everyone.

Browse Source
This commit is contained in:
Doug
2021-12-08 12:00:23 +00:00
parent 70e0eed440
commit ecfaed87b7
6 changed files with 26 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
RiotSwiftUI/Modules/AnalyticsPrompt/View/AnalyticsPrompt.swift
View File

@@ -40,6 +40,8 @@ struct AnalyticsPrompt: View {
Text("\(viewModel.viewState.promptType.description)\n")
AnalyticsPromptTermsText(attributedString: viewModel.viewState.promptType.termsStrings)
.accessibilityLabel(Text(viewModel.viewState.promptType.termsStrings.string))
.accessibilityValue(Text(VectorL10n.accessibilityButtonLabel))
.onTapGesture {
viewModel.send(viewAction: .openTermsURL)
}
@@ -50,10 +52,15 @@ struct AnalyticsPrompt: View {
private var checkmarkList: some View {
VStack(alignment: .leading) {
AnalyticsPromptCheckmarkItem(attributedString: viewModel.viewState.strings.point1)
.accessibilityLabel(Text(viewModel.viewState.strings.point1.string))
AnalyticsPromptCheckmarkItem(attributedString: viewModel.viewState.strings.point2)
.accessibilityLabel(Text(viewModel.viewState.strings.point2.string))
AnalyticsPromptCheckmarkItem(string: VectorL10n.analyticsPromptPoint3)
}
.font(theme.fonts.body)
.frame(maxWidth: .infinity)
}
/// The stack of enable/disable buttons.
@@ -89,6 +96,7 @@ struct AnalyticsPrompt: View {
.padding(.bottom, 2)
descriptionText
.font(theme.fonts.body)
.foregroundColor(theme.colors.secondaryContent)
.multilineTextAlignment(.center)