Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Clarify screenshot content descriptions

Browse Source 
The accessibility descriptions are unclear since they just say
"share" or "edit"; update to say "share screenshot"/"edit
screenshot".

Bug: 200960886
Fix: 200960886
Test: manual (with talkback on)

Change-Id: I544f5db062b717e637f215984532c8f81f0c7eb1
This commit is contained in:
Miranda Kephart
2021-10-26 12:44:23 -04:00
parent d98d6ce999
commit 68cbaa4775
3 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
packages/SystemUI/res/layout/screenshot_static.xml
View File

@@ -94,7 +94,7 @@
android:layout_height="wrap_content" android:layout_height="wrap_content"
android:layout_gravity="center" android:layout_gravity="center"
android:elevation="@dimen/screenshot_preview_elevation" android:elevation="@dimen/screenshot_preview_elevation"
android:contentDescription="@string/screenshot_edit_label" android:contentDescription="@string/screenshot_edit_description"
android:scaleType="fitEnd" android:scaleType="fitEnd"
android:background="@drawable/screenshot_preview_background" android:background="@drawable/screenshot_preview_background"
android:adjustViewBounds="true" android:adjustViewBounds="true"

3
packages/SystemUI/res/values/strings.xml
View File

@@ -183,6 +183,9 @@
<!-- Label for UI element which allows editing the screenshot [CHAR LIMIT=30] --> <!-- Label for UI element which allows editing the screenshot [CHAR LIMIT=30] -->
<string name="screenshot_edit_label">Edit</string> <string name="screenshot_edit_label">Edit</string>
<!-- Content description indicating that tapping the element will allow editing the screenshot [CHAR LIMIT=NONE] --> <!-- Content description indicating that tapping the element will allow editing the screenshot [CHAR LIMIT=NONE] -->
<string name="screenshot_edit_description">Edit screenshot</string>
<!-- Content description indicating that tapping the element will allow sharing the screenshot [CHAR LIMIT=NONE] -->
<string name="screenshot_share_description">Share screenshot</string>
<!-- Label for UI element which allows the user to capture additional off-screen content in a screenshot. [CHAR LIMIT=30] --> <!-- Label for UI element which allows the user to capture additional off-screen content in a screenshot. [CHAR LIMIT=30] -->
<string name="screenshot_scroll_label">Capture more</string> <string name="screenshot_scroll_label">Capture more</string>
<!-- Content description indicating that tapping a button will dismiss the screenshots UI [CHAR LIMIT=NONE] --> <!-- Content description indicating that tapping a button will dismiss the screenshots UI [CHAR LIMIT=NONE] -->

4
packages/SystemUI/src/com/android/systemui/screenshot/ScreenshotView.java
View File

@@ -622,7 +622,7 @@ public class ScreenshotView extends FrameLayout implements
ArrayList<ScreenshotActionChip> chips = new ArrayList<>(); ArrayList<ScreenshotActionChip> chips = new ArrayList<>();
mShareChip.setContentDescription(mContext.getString(com.android.internal.R.string.share)); mShareChip.setContentDescription(mContext.getString(R.string.screenshot_share_description));
mShareChip.setIcon(Icon.createWithResource(mContext, R.drawable.ic_screenshot_share), true); mShareChip.setIcon(Icon.createWithResource(mContext, R.drawable.ic_screenshot_share), true);
mShareChip.setOnClickListener(v -> { mShareChip.setOnClickListener(v -> {
mShareChip.setIsPending(true); mShareChip.setIsPending(true);
@@ -634,7 +634,7 @@ public class ScreenshotView extends FrameLayout implements
}); });
chips.add(mShareChip); chips.add(mShareChip);
mEditChip.setContentDescription(mContext.getString(R.string.screenshot_edit_label)); mEditChip.setContentDescription(mContext.getString(R.string.screenshot_edit_description));
mEditChip.setIcon(Icon.createWithResource(mContext, R.drawable.ic_screenshot_edit), true); mEditChip.setIcon(Icon.createWithResource(mContext, R.drawable.ic_screenshot_edit), true);
mEditChip.setOnClickListener(v -> { mEditChip.setOnClickListener(v -> {
mEditChip.setIsPending(true); mEditChip.setIsPending(true);