Bug fix: screenreader rereading whole page. Improve a11y eqn preview

IM-871, IM-872

Author: drlippman

assess2/index.php

@@ -2,7 +2,7 @@
 // IMathAS: Main launch page for assess2 assessment player
 // (c) 2019 David Lippman
 
-$lastupdate = '20200521';
+$lastupdate = '20200525';
 
 require('../init.php');
 if (empty($_GET['cid']) || empty($_GET['aid'])) {
@@ -36,7 +36,7 @@
 $placeinhead .= '<link rel="stylesheet" type="text/css" href="'.$imasroot.'/assess2/vue/css/chunk-common.css?v='.$lastupdate.'" />';
 $placeinhead .= '<link rel="stylesheet" type="text/css" href="'.$imasroot.'/assess2/print.css?v='.$lastupdate.'" media="print">';
 $placeinhead .= '<script src="'.$imasroot.'/mathquill/mathquill.min.js?v=022720" type="text/javascript"></script>';
-$placeinhead .= '<script src="'.$imasroot.'/javascript/assess2_min.js?v=052120" type="text/javascript"></script>';
+$placeinhead .= '<script src="'.$imasroot.'/javascript/assess2_min.js?v=052520" type="text/javascript"></script>';
 $placeinhead .= '<link rel="stylesheet" type="text/css" href="'.$imasroot.'/mathquill/mathquill-basic.css">
   <link rel="stylesheet" type="text/css" href="'.$imasroot.'/mathquill/mqeditor.css">';
 if ($isltilimited || $inTreeReader) {

assess2/vue-src/public/index.html

@@ -10,7 +10,16 @@
     <script type="text/javascript">
       var noMathRender = false; var usingASCIIMath = true; var AMnoMathML = true;
       var MathJaxCompatible = true; var mathRenderer="MathJax";
-      function rendermathnode(node) { MathJax.Hub.Queue(["Typeset", MathJax.Hub, node]); }
+      function rendermathnode(node,callback) {
+  			if (window.MathJax) {
+  				MathJax.Hub.Queue(["Typeset", MathJax.Hub, node]);
+  				if (typeof callback == "function") {
+  					MathJax.Hub.Queue(callback);
+  				}
+  			} else {
+  				setTimeout(function() {rendermathnode(node, callback);}, 100);
+  			}
+      }
       var imasroot = 'http://localhost/imathas';
       var APIbase = 'http://localhost/imathas/assess2/';
       var coursetheme = "https://ohm.lumenlearning.com/themes/lumen.css?v=110918";

assess2/vue-src/src/App.vue

@@ -1,5 +1,5 @@
 <template>
-  <div id="app" role="main" aria-live="polite">
+  <div id="app" role="main">
     <div v-if="!assessInfoLoaded">
       {{ $t('loading') }}
     </div>

assess2/vue-src/src/basicstore.js

@@ -506,6 +506,7 @@ export const actions = {
             var el;
             if (!hasSeqNext) {
               el = document.getElementById('questionwrap' + qns[0]).parentNode.parentNode;
+              window.$(el).find('.scoreresult').focus();
             } else {
               el = window.$('#questionwrap' + qns[0]).find('.seqsep').last().next()[0];
             }

assess2/vue-src/src/components/question/QuestionHelps.vue

@@ -5,7 +5,7 @@
     </li>
     <li v-for="(qHelp,idx) in qHelps" :key="idx">
       <a href="#" @click.prevent="loadHelp(qHelp)">
-        <icons :name="qHelp.icon" />
+        <icons :name="qHelp.icon" alt=""/>
         {{ qHelp.title }}
       </a>
     </li>

assess2/vue-src/src/components/question/ScoreResult.vue

@@ -2,6 +2,7 @@
   <transition name="fade">
     <div
       :class="['scoreresult', status]"
+      tabindex = "-1"
       v-if="expanded"
     >
       <p v-if="showScores">
@@ -24,9 +25,8 @@
           v-if = "showNext"
           :to="'/skip/' + (this.qn + 2)"
           tag="button"
-          :aria-label="$t('next')"
         >
-          <icons name="right" />
+          <icons name="right" alt=""/>
           {{ $t('scoreresult.next') }}
         </router-link>
         <button
@@ -42,7 +42,7 @@
           type = "button"
           @click = "trySimilar"
         >
-          <icons name="retake" />
+          <icons name="retake" alt="" />
           {{ $t('scoreresult.trysimilar') }}
         </button>
         <span v-if = "qdata.canretry">

assess2/vue-src/src/components/widgets/Icons.vue

@@ -157,7 +157,7 @@ export default {
   props: ['name', 'size', 'color', 'alt'],
   computed: {
     alttext () {
-      if (this.alt) {
+      if (this.alt || this.alt === '') {
         if (this.alt === '') {
           return '';
         } else {