Ensures compatibility of in-place speech computations.

Author: zorkow

ts/a11y/explorer/KeyExplorer.ts

@@ -28,7 +28,7 @@ import type { ExplorerMathItem } from '../explorer.js';
 import {Explorer, AbstractExplorer} from './Explorer.js';
 import {ExplorerPool} from './ExplorerPool.js';
 import {MmlNode} from '../../core/MmlTree/MmlNode.js';
-import { honk } from '../speech/SpeechUtil.js';
+import { honk, InPlace } from '../speech/SpeechUtil.js';
 import {Sre} from '../sre.js';
 
 
@@ -312,6 +312,7 @@ export class SpeechExplorer extends AbstractExplorer<string> implements KeyExplo
   public depth(node: HTMLElement): HTMLElement {
     this.generators.depth(node, !!this.actionable(node));
     this.refocus(node);
+    this.generators.lastMove = InPlace.DEPTH;
     return node;
   }
 
@@ -337,6 +338,7 @@ export class SpeechExplorer extends AbstractExplorer<string> implements KeyExplo
   public summary(node: HTMLElement): HTMLElement {
     this.generators.summary(node);
     this.refocus(node);
+    this.generators.lastMove = InPlace.SUMMARY;
     return node;
   }
 

ts/a11y/speech/GeneratorPool.ts

@@ -19,7 +19,7 @@ import {mathjax} from '../../mathjax.js';
 import {Sre} from '../sre.js';
 import {OptionList} from '../../util/Options.js';
 import {LiveRegion} from '../explorer/Region.js';
-import { buildLabel, buildSpeech } from '../speech/SpeechUtil.js';
+import { buildLabel, buildSpeech, InPlace } from '../speech/SpeechUtil.js';
 import { DOMAdaptor } from '../../core/DOMAdaptor.js';
 
 /**
@@ -177,7 +177,7 @@ export class GeneratorPool<N, T, D> {
    * @param {N} node The typeset node.
    */
   public summary(node: N) {
-    if (this.lastSummary) {
+    if (this.lastMove === InPlace.SUMMARY) {
       this.CleanUp(node);
       return this.lastSpeech;
     }
@@ -193,11 +193,11 @@ export class GeneratorPool<N, T, D> {
    * @param {N} node
    */
   public CleanUp(node: N) {
-    if (this.lastSummary) {
+    if (this.lastMove) {
       // TODO: Remember the speech.
       this.adaptor.setAttribute(node, 'aria-label', buildSpeech(this.getLabel(node))[0]);
     }
-    this.lastSummary = false;
+    this.lastMove = InPlace.NONE;
   }
 
   /**
@@ -206,9 +206,24 @@ export class GeneratorPool<N, T, D> {
   private lastSpeech = '';
 
   /**
-   * Remembers that the last speech computation was a summary.
+   * Remembers whether the last speech computation was in-place, i.e., a summary
+   * or depth computation.
    */
-  private lastSummary = false;
+  private lastMove_ = InPlace.NONE;
+
+  /**
+   * Getter for last move.
+   */
+  public get lastMove() {
+    return this.lastMove_;
+  }
+
+  /**
+   * Setter for last move.
+   */
+  public set lastMove(move: InPlace) {
+    this.lastMove_ = this.lastSpeech ? move : InPlace.NONE;
+  }
 
   /**
    * Updates the given speech regions, possibly reinstanting previously saved
@@ -225,11 +240,7 @@ export class GeneratorPool<N, T, D> {
   ) {
     let speech = this.getLabel(node, this.lastSpeech);
     speechRegion.Update(speech);
-    // TODO: See if we can reuse the speech from the speech region.
     this.adaptor.setAttribute(node, 'aria-label', buildSpeech(speech)[0]);
-    if (this.lastSpeech) {
-      this.lastSummary = true;
-    }
     this.lastSpeech = '';
     brailleRegion.Update(
       this.adaptor.getAttribute(node, 'aria-braillelabel'));
@@ -379,7 +390,7 @@ export class GeneratorPool<N, T, D> {
   }
 
   public depth(node: N, actionable: boolean) {
-    if (this.lastSummary) {
+    if (this.lastMove === InPlace.DEPTH) {
       this.CleanUp(node);
       return this.lastSpeech;
     }

ts/a11y/speech/SpeechUtil.ts

@@ -213,3 +213,13 @@ export function honk() {
   os.start(ac.currentTime);
   os.stop(ac.currentTime + .05);
 }
+
+/**
+ * In place speech computations.
+ */
+export enum InPlace {
+  NONE,
+  DEPTH,
+  SUMMARY
+}
+