From ed77f97283eb4de1f5c57555a7426593c7b2128d Mon Sep 17 00:00:00 2001
From: Eitan Isaacson <eitan@monotonous.org>
Date: Tue, 2 Apr 2019 15:00:00 -0700
Subject: [PATCH] Remove Quick Actions Sheet contentDescription

This is not needed for screen reader usability, and introduces an
useless intermediate node that TalkBack lands on.
---
 app/src/main/res/layout/layout_quick_action_sheet.xml | 1 -
 app/src/main/res/values/strings.xml                   | 2 --
 2 files changed, 3 deletions(-)

diff --git a/app/src/main/res/layout/layout_quick_action_sheet.xml b/app/src/main/res/layout/layout_quick_action_sheet.xml
index ebbe13231..14e7b7548 100644
--- a/app/src/main/res/layout/layout_quick_action_sheet.xml
+++ b/app/src/main/res/layout/layout_quick_action_sheet.xml
@@ -9,7 +9,6 @@
     android:orientation="vertical"
     android:layout_width="match_parent"
     android:layout_height="wrap_content"
-    android:contentDescription="@string/quick_action_sheet"
     android:background="?attr/toolbarColor">
 
     <androidx.appcompat.widget.AppCompatImageButton
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 70612faf4..e18aad9e4 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -177,8 +177,6 @@
     <string name="quick_action_bookmark">Bookmark</string>
     <!-- Option in Quick Action Sheet in the browser to put the the current page in reader mode -->
     <string name="quick_action_read">Read</string>
-    <!-- Content description (not visible, for screen readers etc.): Quick action sheet-->
-    <string name="quick_action_sheet">Quick action sheet</string>
     <!-- Content description (not visible, for screen readers etc.): Quick action sheet handle -->
     <string name="quick_action_sheet_handle">Quick actions</string>