Skip to content
Snippets Groups Projects
Unverified Commit b90b2957 authored by harambasicluka's avatar harambasicluka Committed by GitHub
Browse files

Fix: a18y (#413)

* image descriptions

* brought the main overview back

* added "Button" to submission dispatcher cards

* fix button icon a18y, button is now also announced
parent 5e9dd007
No related branches found
No related tags found
No related merge requests found
Showing
with 47 additions and 9 deletions
......@@ -93,10 +93,10 @@
<TextView
style="@style/headline6"
android:accessibilityHeading="true"
android:layout_width="0dp"
android:layout_height="wrap_content"
android:layout_marginStart="@dimen/guideline_body_title"
android:accessibilityHeading="true"
android:text="@string/main_overview_subtitle_risk_levels"
app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintStart_toStartOf="parent"
......@@ -174,10 +174,10 @@
<TextView
style="@style/headline5"
android:accessibilityHeading="true"
android:layout_width="0dp"
android:layout_height="wrap_content"
android:layout_marginStart="@dimen/guideline_body_title"
android:accessibilityHeading="true"
android:text="@string/main_overview_headline_glossary"
app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintStart_toStartOf="parent"
......
......@@ -9,6 +9,8 @@
<import type="de.rki.coronawarnapp.util.formatter.FormatterRiskHelper" />
<import type="de.rki.coronawarnapp.util.formatter.FormatterAccessibilityHelper" />
<import type="android.view.View" />
<variable
......@@ -49,7 +51,7 @@
style="@style/icon"
android:layout_width="@dimen/icon_size_button"
android:layout_height="@dimen/icon_size_button"
android:contentDescription="@{@string/accessibility_back}"
android:contentDescription="@{FormatterAccessibilityHelper.formatButton(@string/accessibility_back)}"
android:src="@{@drawable/ic_close}"
android:tint="@{FormatterRiskHelper.formatStableTextColor(tracingViewModel.riskLevel)}"
app:layout_constraintBottom_toBottomOf="parent"
......
......@@ -2,6 +2,12 @@
<layout xmlns:android="http://schemas.android.com/apk/res/android"
xmlns:app="http://schemas.android.com/apk/res-auto">
<data>
<import type="de.rki.coronawarnapp.util.formatter.FormatterAccessibilityHelper" />
</data>
<androidx.constraintlayout.widget.ConstraintLayout
android:id="@+id/submission_qr_code_scan_container"
android:contentDescription="@string/submission_qr_code_scan_title"
......@@ -68,7 +74,7 @@
style="@style/iconStable"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:contentDescription="@string/accessibility_close"
android:contentDescription="@{FormatterAccessibilityHelper.formatImage(@string/accessibility_close)"
android:src="@drawable/ic_close"
app:layout_constraintBottom_toBottomOf="parent"
app:layout_constraintEnd_toEndOf="parent"
......
......@@ -21,6 +21,7 @@
<androidx.constraintlayout.widget.ConstraintLayout
android:id="@+id/button_icon"
style="@style/buttonIcon"
android:contentDescription="@{FormatterAccessibilityHelper.formatButton(iconDescription)}"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
app:layout_constraintStart_toStartOf="parent"
......@@ -31,7 +32,7 @@
android:layout_width="@dimen/icon_size_button"
android:layout_height="@dimen/icon_size_button"
android:src="@{icon}"
android:contentDescription="@{FormatterAccessibilityHelper.formatButton(iconDescription)}"
android:importantForAccessibility="no"
app:layout_constraintBottom_toBottomOf="parent"
app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintStart_toStartOf="parent"
......
......@@ -5,6 +5,8 @@
<data>
<import type="de.rki.coronawarnapp.util.formatter.FormatterAccessibilityHelper" />
<variable
name="headline"
type="String" />
......@@ -50,6 +52,7 @@
android:layout_marginEnd="@dimen/spacing_normal"
android:src="@drawable/ic_forward"
android:tint="@color/colorTextPrimary1"
android:importantForAccessibility="no"
app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintTop_toTopOf="parent" />
......@@ -63,6 +66,7 @@
android:layout_marginEnd="@dimen/spacing_small"
android:layout_marginBottom="@dimen/spacing_normal"
android:text="@{body}"
android:contentDescription="@{FormatterAccessibilityHelper.formatButton(body)}"
app:layout_constraintBottom_toBottomOf="parent"
app:layout_constraintEnd_toStartOf="@+id/dispatcher_card_illustration"
app:layout_constraintStart_toStartOf="parent"
......@@ -76,6 +80,7 @@
android:layout_height="wrap_content"
android:layout_marginTop="@dimen/button_icon_margin"
android:src="@{illustration}"
android:importantForAccessibility="no"
app:layout_constraintBottom_toBottomOf="parent"
app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintTop_toBottomOf="@+id/dispatcher_card_icon"
......
......@@ -2,6 +2,12 @@
<layout xmlns:android="http://schemas.android.com/apk/res/android"
xmlns:app="http://schemas.android.com/apk/res-auto">
<data>
<import type="de.rki.coronawarnapp.util.formatter.FormatterAccessibilityHelper" />
</data>
<ScrollView
android:layout_width="match_parent"
android:layout_height="match_parent">
......@@ -15,7 +21,7 @@
android:layout_width="@dimen/match_constraint"
android:layout_height="wrap_content"
android:src="@drawable/ic_submission_illustration_hotline"
android:contentDescription="@string/information_contact_illustration_description"
android:contentDescription="@{FormatterAccessibilityHelper.formatImage(@string/information_contact_illustration_description)}"
android:focusable="true"
app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintStart_toStartOf="parent"
......
......@@ -2,6 +2,12 @@
<layout xmlns:android="http://schemas.android.com/apk/res/android"
xmlns:app="http://schemas.android.com/apk/res-auto">
<data>
<import type="de.rki.coronawarnapp.util.formatter.FormatterAccessibilityHelper" />
</data>
<ScrollView
android:layout_width="match_parent"
android:layout_height="match_parent">
......@@ -15,7 +21,7 @@
android:id="@+id/submission_done_hero_illustration"
android:layout_width="@dimen/match_constraint"
android:layout_height="wrap_content"
android:contentDescription="@string/submission_done_illustration_description"
android:contentDescription="@{FormatterAccessibilityHelper.formatImage(@string/submission_done_illustration_description)}"
android:focusable="true"
android:src="@drawable/ic_submission_illustration_thanks"
app:layout_constraintEnd_toEndOf="parent"
......
......@@ -2,6 +2,12 @@
<layout xmlns:android="http://schemas.android.com/apk/res/android"
xmlns:app="http://schemas.android.com/apk/res-auto">
<data>
<import type="de.rki.coronawarnapp.util.formatter.FormatterAccessibilityHelper" />
</data>
<ScrollView
android:layout_width="match_parent"
android:layout_height="match_parent">
......@@ -12,7 +18,7 @@
<ImageView
android:id="@+id/submission_intro_hero_illustration"
android:contentDescription="@{@string/submission_intro_illustration_description}"
android:contentDescription="@{FormatterAccessibilityHelper.formatImage(@string/submission_intro_illustration_description)}"
android:layout_width="@dimen/match_constraint"
android:layout_height="wrap_content"
android:src="@drawable/ic_illustration_test"
......
......@@ -94,6 +94,7 @@
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:background="@{FormatterSubmissionHelper.formatSubmissionStatusCardContentIcon(deviceUIState)}"
android:importantForAccessibility="no"
app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintTop_toTopOf="@id/submission_status_card_content_title"
tools:src="@drawable/ic_main_illustration_negative" />
......
......@@ -59,6 +59,7 @@
android:id="@+id/submission_status_card_unregistered_icon"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:importantForAccessibility="no"
app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintTop_toTopOf="@+id/submission_status_card_unregistered_title"
app:srcCompat="@drawable/ic_main_illustration_untested" />
......
......@@ -54,6 +54,7 @@
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:background="@{FormatterSubmissionHelper.formatTestStatusIcon(deviceUIState)}"
android:importantForAccessibility="no"
app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintTop_toTopOf="parent"
tools:src="@drawable/ic_test_result_illustration_negative" />
......
......@@ -25,6 +25,7 @@
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:background="@drawable/ic_test_result_illustration_positive_card"
android:importantForAccessibility="no"
app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintTop_toTopOf="parent"
tools:src="@drawable/ic_test_result_illustration_positive" />
......
......@@ -13,6 +13,7 @@
android:layout_marginStart="@dimen/bullet_point_spacing_before"
android:baseline="@dimen/bullet_point_baseline_offset"
android:src="@drawable/bullet_point"
android:importantForAccessibility="no"
app:layout_constraintBaseline_toBaselineOf="@+id/bullet_point_content"
app:layout_constraintStart_toStartOf="parent" />
......
......@@ -20,7 +20,7 @@
android:id="@+id/circle_progress_icon"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:contentDescription="Sie haben alle tage erfasst"
android:importantForAccessibility="no"
android:src="@drawable/ic_risk_card_saved_days"
android:tint="@color/colorAccentTintIcon"
app:layout_constraintBottom_toBottomOf="parent"
......
......@@ -11,6 +11,7 @@
android:id="@+id/step_entry_icon"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:importantForAccessibility="no"
app:layout_constraintStart_toStartOf="parent"
app:layout_constraintTop_toTopOf="parent"
tools:src="@drawable/ic_main_overview_1" />
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment