Add missing PDF reader labels

Affects: https://ebce-lyrasis.atlassian.net/browse/PP-4021

Author: io7m

org.thepalaceproject.android.platform

@@ -20,17 +20,22 @@
             android:layout_height="64dp"
             android:background="?attr/selectableItemBackground"
             android:clickable="true"
+            android:contentDescription="@string/accessibility_back"
+            android:focusable="true"
             app:layout_constraintBottom_toBottomOf="parent"
             app:layout_constraintStart_toStartOf="parent"
             app:layout_constraintTop_toTopOf="parent">
 
             <ImageView
-                app:tint="@color/PalacePrimaryForegroundColor"
                 android:id="@+id/pdfToolbarLogo"
                 android:layout_width="24dp"
                 android:layout_height="24dp"
                 android:layout_gravity="center"
-                android:src="@drawable/arrow_left" />
+                android:clickable="false"
+                android:focusable="false"
+                android:importantForAccessibility="no"
+                android:src="@drawable/arrow_left"
+                app:tint="@color/PalacePrimaryForegroundColor" />
         </FrameLayout>
 
         <TextView
@@ -53,6 +58,8 @@
             android:layout_height="64dp"
             android:background="?attr/selectableItemBackground"
             android:clickable="true"
+            android:contentDescription="@string/accessibility_toc"
+            android:focusable="true"
             app:layout_constraintBottom_toBottomOf="parent"
             app:layout_constraintEnd_toStartOf="@id/pdfToolbarSettingsIconTouch"
             app:layout_constraintTop_toTopOf="parent">
@@ -62,6 +69,9 @@
                 android:layout_width="24dp"
                 android:layout_height="24dp"
                 android:layout_gravity="center"
+                android:clickable="false"
+                android:focusable="false"
+                android:importantForAccessibility="no"
                 android:src="@drawable/pdf_toc" />
         </FrameLayout>
 
@@ -71,6 +81,8 @@
             android:layout_height="64dp"
             android:background="?attr/selectableItemBackground"
             android:clickable="true"
+            android:contentDescription="@string/accessibility_settings"
+            android:focusable="true"
             app:layout_constraintBottom_toBottomOf="parent"
             app:layout_constraintEnd_toEndOf="parent"
             app:layout_constraintTop_toTopOf="parent">
@@ -80,6 +92,9 @@
                 android:layout_width="24dp"
                 android:layout_height="24dp"
                 android:layout_gravity="center"
+                android:clickable="false"
+                android:focusable="false"
+                android:importantForAccessibility="no"
                 android:src="@drawable/pdf_settings" />
         </FrameLayout>
 
@@ -98,6 +113,9 @@
             android:layout_width="128dp"
             android:layout_height="32dp"
             android:layout_gravity="center"
+            android:clickable="false"
+            android:focusable="false"
+            android:importantForAccessibility="no"
             android:indeterminate="true" />
 
     </FrameLayout>

palace-viewer-pdf-pdfjs/src/main/res/layout/pdfjs_reader.xml

@@ -1,17 +1,16 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 
 <resources>
-  <string name="Dismiss">Dismiss</string>
-
-  <string name="accessibility_table_of_contents">Open the table of contents</string>
-  <string name="table_of_contents_title">Table of Contents</string>
-  <string name="table_of_contents_empty_message">No Table of Contents available for this work</string>
-  <string name="accessibility_settings">Open the settings menu</string>
-  <string name="settings_title">Settings</string>
-  <string name="content_description_back">Back</string>
-
-  <string name="viewer_position_title">Sync Reading Position</string>
-  <string name="viewer_position_message">Do you want to move to the page on which you left off?</string>
-  <string name="viewer_position_move">Move</string>
-  <string name="viewer_position_stay">Stay</string>
+    <string name="Dismiss">Dismiss</string>
+    <string name="accessibility_back">Close book and go back.</string>
+    <string name="accessibility_settings">Open the settings menu</string>
+    <string name="accessibility_toc">Open the table of contents and bookmarks screen.</string>
+    <string name="content_description_back">Back</string>
+    <string name="settings_title">Settings</string>
+    <string name="table_of_contents_empty_message">No Table of Contents available for this work</string>
+    <string name="table_of_contents_title">Table of Contents</string>
+    <string name="viewer_position_message">Do you want to move to the page on which you left off?</string>
+    <string name="viewer_position_move">Move</string>
+    <string name="viewer_position_stay">Stay</string>
+    <string name="viewer_position_title">Sync Reading Position</string>
 </resources>