Merge pull request #1536 from dimagi/em/clickable-icons-alt-text

Alt text for icons

Author: nospame

Diff for: libs/commcare

@@ -10,6 +10,7 @@ public class EntityBean {
     private Object[] data;
 
     private String groupKey;
+    private String[] altText;
 
     public EntityBean() {
     }
@@ -42,6 +43,14 @@ public void setGroupKey(String groupKey) {
         this.groupKey = groupKey;
     }
 
+    public String[] getAltText() {
+        return altText;
+    }
+
+    public void setAltText(String[] altText) {
+        this.altText = altText;
+    }
+
     @Override
     public String toString() {
         return "EntityBean with id=" + id + ", data=" + Arrays.toString(data);

Diff for: src/main/java/org/commcare/formplayer/beans/menus/EntityBean.java

@@ -33,6 +33,7 @@ public class EntityDetailResponse {
     private EntityBean[] entities;
     protected Style[] styles;
     protected String[] headers;
+    protected String[] altText;
     protected String title;
     protected boolean isUseNodeset;
 
@@ -53,6 +54,7 @@ public EntityDetailResponse(EntityDetailSubscreen entityScreen, String title) {
         this.headers = entityScreen.getHeaders();
         this.styles = entityScreen.getStyles();
         this.tiles = processCaseTiles(entityScreen.getDetail());
+        this.altText = entityScreen.getAltText();
     }
 
     private static Object[] processDetails(Object[] data) {
@@ -173,6 +175,14 @@ public void setTitle(String title) {
         this.title = title;
     }
 
+    public String[] getAltText() {
+        return altText;
+    }
+
+    public void setAltText(String[] altText) {
+        this.altText = altText;
+    }
+
     public boolean isUsesCaseTiles() {
         return usesCaseTiles;
     }

Diff for: src/main/java/org/commcare/formplayer/beans/menus/EntityDetailResponse.java

@@ -245,6 +245,7 @@ private static EntityBean toEntityBean(Entity<TreeReference> entity,
         }
         entityBean.setData(data);
         entityBean.setGroupKey(entity.getGroupKey());
+        entityBean.setAltText(entity.getAltText());
         return entityBean;
     }
 

Diff for: src/main/java/org/commcare/formplayer/beans/menus/EntityListResponse.java

@@ -8,6 +8,7 @@ import org.commcare.formplayer.junit.*
 import org.commcare.formplayer.junit.Installer.Companion.getInstallReference
 import org.commcare.formplayer.junit.request.SessionNavigationRequest
 import org.commcare.formplayer.utils.TestContext
+import org.junit.jupiter.api.Assertions.assertArrayEquals
 import org.junit.jupiter.api.Assertions.assertEquals
 import org.junit.jupiter.api.Test
 import org.junit.jupiter.api.extension.ExtendWith
@@ -55,6 +56,7 @@ class CaseListLazyLoadingTests {
         assertEquals(singleEntity.data[0], "Batman Begins")
         assertEquals(singleEntity.data[1], "Batman Begins")
         assertEquals(singleEntity.groupKey, "Batman Begins")
+        assertArrayEquals(singleEntity.altText, arrayOfNulls<String>(singleEntity.data.size))
 
         response = navigate(selections, EntityListResponse::class.java, 1, 3)
         entitites = response.entities
@@ -64,6 +66,7 @@ class CaseListLazyLoadingTests {
         assertEquals(singleEntity.data[0], "Rudolph")
         assertEquals(singleEntity.data[1], "Rudolph")
         assertEquals(singleEntity.groupKey, "Rudolph")
+        assertArrayEquals(singleEntity.altText, arrayOfNulls<String>(singleEntity.data.size))
     }
 
     private fun <T : BaseResponseBean> navigate(