incorporate review suggestions

Author: zorkow

ts/input/tex/textmacros/TextMacrosConfiguration.ts

@@ -58,7 +58,7 @@ export const TextBaseConfiguration = Configuration.create('text-base', {
     //   produces the error as configured in the main TeX parser, so
     //   this will respect the noundefined package, if loaded).
     //
-    macro: (parser: TextParser, name: string) => {
+    [HandlerType.MACRO]: (parser: TextParser, name: string) => {
       const texParser = parser.texParser;
       const macro = texParser.lookup(HandlerType.MACRO, name);
       if (macro && macro._func !== TextMacrosMethods.Macro) {
@@ -71,7 +71,7 @@ export const TextBaseConfiguration = Configuration.create('text-base', {
       texParser.parse(HandlerType.MACRO, [parser, name]);
     },
   },
-  items: {
+  [ConfigurationType.ITEMS]: {
     [StartItem.prototype.kind]: StartItem,
     [StopItem.prototype.kind]: StopItem,
     [MmlItem.prototype.kind]: MmlItem,
@@ -118,7 +118,10 @@ export const TextMacrosConfiguration = Configuration.create('textmacros', {
    * @param {ParserConfiguration} _config  The configuration object we are being configured within
    * @param {TeX<any,any,any>} jax         The TeX input jax in which we are running
    */
-  config(_config: ParserConfiguration, jax: TeX<any, any, any>) {
+  [ConfigurationType.CONFIG]: (
+    _config: ParserConfiguration,
+    jax: TeX<any, any, any>
+  ) => {
     //
     //  Create the configuration and parseOptions objects for the
     //    internal TextParser and add the textBase configuration.

ts/output/common.ts

@@ -215,7 +215,7 @@ export abstract class CommonOutputJax<
   >;
 
   /**
-   * @returns {boolean} True when inkline breaks need to be forced (e.g., for
+   * @returns {boolean} True when in-line breaks need to be forced (e.g., for
    *     SVG output)
    */
   get forceInlineBreaks(): boolean {
@@ -676,7 +676,7 @@ export abstract class CommonOutputJax<
 
   /**
    * @param {N} node    The math element to be measured
-   * @param {boolean} display Is the element is display math?
+   * @param {boolean} display Is the element in display math?
    * @returns {N}        The test elements that were added
    */
   protected getTestElement(node: N, display: boolean): N {

ts/output/common/Notation.ts

@@ -118,11 +118,11 @@ export const sideNames = Object.keys(sideIndex) as Side[];
  * @param {Menclose} node The enclose node
  * @returns {BBoxExtender<Menclose>} The bbox extender
  */
-export const fullBBox = ((node: Menclose) =>
+export const fullBBox = ((node) =>
   new Array(4).fill(node.thickness + node.padding)) as BBoxExtender<Menclose>;
-export const fullPadding = ((node: Menclose) =>
+export const fullPadding = ((node) =>
   new Array(4).fill(node.padding)) as BBoxExtender<Menclose>;
-export const fullBorder = ((node: Menclose) =>
+export const fullBorder = ((node) =>
   new Array(4).fill(node.thickness)) as BBoxBorder<Menclose>;
 
 /*****************************************************************/

ts/output/common/Wrappers/XmlNode.ts

@@ -278,7 +278,7 @@ export function CommonXmlNodeMixin<
     /**
      * The font-size and font-family values to use for the XML
      *
-     * @returns {{'font-family': string, 'font-size': string}} The font info
+     * @returns {StyleList} The font info
      */
     public getFontStyles(): { 'font-family': string; 'font-size': string } {
       const adaptor = this.adaptor;

ts/ui/menu/Menu.ts

@@ -1105,7 +1105,7 @@ export class Menu {
   /**
    * Add extensions that need to be loaded when the renderer changes
    *
-   * @param {string[] }extensions The list of extensions
+   * @param {string[]} extensions The list of extensions
    */
   public addRequiredExtensions(extensions: string[]) {
     if (extensions) {