From 2e7159b3c92a93c31525289c35d2f5eee0a66459 Mon Sep 17 00:00:00 2001
From: "kirsty.hames" <kirsty.hames@kineo.com>
Date: Mon, 18 Nov 2019 12:52:48 +0000
Subject: [PATCH] tab order and alt text amends

- insert button immediately after back button (so that tab order is correct).
- add button alt text.
- indent example json.
---
 bower.json             |  2 +-
 example.json           | 35 ++++++++++++++++++-----------------
 js/adapt-homeButton.js | 18 ++++++++++++++----
 properties.schema      |  9 +++++++++
 4 files changed, 42 insertions(+), 22 deletions(-)

diff --git a/bower.json b/bower.json
index b425e36..327ddba 100644
--- a/bower.json
+++ b/bower.json
@@ -5,7 +5,7 @@
         "url": "git://github.com/cgkineo/adapt-homeButton.git"
     },
     "framework": ">=2.0.18",
-    "version": "0.0.7",
+    "version": "0.0.8",
     "homepage": "https://github.com/cgkineo/adapt-homeButton",
     "displayName": "Home Button",
     "extension": "homeButton",
diff --git a/example.json b/example.json
index bab5bbc..49be2e7 100644
--- a/example.json
+++ b/example.json
@@ -1,19 +1,20 @@
-// config.json
-"_homeButton": {
-    "_isEnabled": true
-}
+    // config.json
+    "_homeButton": {
+        "_isEnabled": true
+    }
 
-// course.json
-"_homeButton": {
-    "_isEnabled": false,
-    "_hideHomeButton": false,
-    "_redirectToId": ""
-}
+    // course.json
+    "_homeButton": {
+        "_isEnabled": false,
+        "_hideHomeButton": false,
+        "_redirectToId": "",
+        "alt": "Home"
+    }
 
-// contentObjects.json
-"_homeButton": {
-    "_isEnabled": false,
-    "_hideHomeButton": false,
-    "_hideBackButton": false,
-    "_redirectToId": ""
-}
\ No newline at end of file
+    // contentObjects.json
+    "_homeButton": {
+        "_isEnabled": false,
+        "_hideHomeButton": false,
+        "_hideBackButton": false,
+        "_redirectToId": ""
+    }
\ No newline at end of file
diff --git a/js/adapt-homeButton.js b/js/adapt-homeButton.js
index 18808f6..a6c206d 100644
--- a/js/adapt-homeButton.js
+++ b/js/adapt-homeButton.js
@@ -55,10 +55,20 @@ define([
         },
 
         _createHomeButton: function() {
-            $('.navigation-inner').append($('<button>', {
-                attr: { 'data-event': 'homeButton' },
-                'class': 'base navigation-home-button icon icon-home'
-            }));
+            var config = Adapt.course.get("_homeButton");
+            var altText = (config && config.alt);
+            var $backButton = $('button[data-event="backButton"]');
+            var $homeButton = $('<button>', {
+                attr: {
+                    'data-event': 'homeButton'
+                },
+                'class': 'base navigation-home-button icon icon-home',
+                'aria-label': altText
+            });
+
+            // insert immediately after back button (so that tab order is correct)
+            // back button is part of core (navigation.hbs)
+            $homeButton.insertAfter($backButton);
         },
 
         _redirected: function() {
diff --git a/properties.schema b/properties.schema
index aee124b..975b043 100644
--- a/properties.schema
+++ b/properties.schema
@@ -59,6 +59,15 @@
                   "title": "Redirect the home button to id",
                   "inputType": "Text",
                   "validators": []
+                },
+                "alt": {
+                  "type": "string",
+                  "required": false,
+                  "default": "Home",
+                  "title": "Home button alt text",
+                  "inputType": "Text",
+                  "validators": [],
+                  "translatable": true
                 }
               }
             }