Adjust comments identified in review.

Author: dpvc

ts/output/svg/Wrappers/mo.ts

@@ -266,7 +266,7 @@ export const SvgMo = (function <N, T, D>(): SvgMoClass<N, T, D> {
     /**
      * @param {number} n         The number of the character to look up
      * @param {string} variant   The variant for the character to look up
-     * @returns {SvgCharData}     The full CharData object, with CharOptions guaranteed to be defined
+     * @returns {SvgCharData}    The full CharData object, with CharOptions guaranteed to be defined
      */
     protected getChar(n: number, variant: string): SvgCharData {
       const char = this.font.getChar(variant, n) || [0, 0, 0, null];
@@ -284,7 +284,7 @@ export const SvgMo = (function <N, T, D>(): SvgMoClass<N, T, D> {
      * @param {number} x         The x position of the glyph
      * @param {number} y         The y position of the glyph
      * @param {N} parent         The container for the glyph
-     * @returns {number}          The width of the character placed
+     * @returns {number}         The width of the character placed
      */
     protected addGlyph(
       n: number,
@@ -312,7 +312,7 @@ export const SvgMo = (function <N, T, D>(): SvgMoClass<N, T, D> {
      * @param {string} v    The variant for the top glyph
      * @param {number} H    The height of the stretched delimiter
      * @param {number} W    The width of the stretched delimiter
-     * @returns {number}     The total height of the top glyph
+     * @returns {number}    The total height of the top glyph
      */
     protected addTop(n: number, v: string, H: number, W: number): number {
       if (!n) return 0;
@@ -340,15 +340,15 @@ export const SvgMo = (function <N, T, D>(): SvgMoClass<N, T, D> {
       W: number
     ) {
       if (!n) return;
-      T = Math.max(0, T - VFUZZ); //             // A little overlap on top
-      B = Math.max(0, B - VFUZZ); //             // A little overlap on bottom
+      T = Math.max(0, T - VFUZZ); //           A little overlap on top
+      B = Math.max(0, B - VFUZZ); //           A little overlap on bottom
       const adaptor = this.adaptor;
       const [h, d, w] = this.getChar(n, v);
-      const Y = H + D - T - B; //                // The height of the extender
-      const s = (1.5 * Y) / (h + d); //          // Scale height by 1.5 to avoid bad ends
-      //                                         //   (glyphs with rounded or anti-aliased ends don't stretch well,
-      //                                         //    so this makes for sharper ends)
-      const y = (s * (h - d) - Y) / 2; //        // The bottom point to clip the extender
+      const Y = H + D - T - B; //              The height of the extender
+      const s = (1.5 * Y) / (h + d); //        Scale height by 1.5 to avoid bad ends
+      //                                         (glyphs with rounded or anti-aliased ends don't stretch well,
+      //                                          so this makes for sharper ends)
+      const y = (s * (h - d) - Y) / 2; //      The bottom point to clip the extender
       if (Y <= 0) return;
       const svg = this.svg('svg', {
         width: this.fixed(w),
@@ -377,7 +377,7 @@ export const SvgMo = (function <N, T, D>(): SvgMoClass<N, T, D> {
      * @param {string} v    The variant for the bottom glyph
      * @param {number} D    The depth of the stretched delimiter
      * @param {number} W    The width of the stretched delimiter
-     * @returns {number}     The total height of the bottom glyph
+     * @returns {number}    The total height of the bottom glyph
      */
     protected addBot(n: number, v: string, D: number, W: number): number {
       if (!n) return 0;
@@ -405,7 +405,7 @@ export const SvgMo = (function <N, T, D>(): SvgMoClass<N, T, D> {
     /**
      * @param {number} n   The character number for the left glyph of the stretchy character
      * @param {string} v   The variant for the left glyph
-     * @returns {number}    The width of the left glyph
+     * @returns {number}   The width of the left glyph
      */
     protected addLeft(n: number, v: string): number {
       return n ? this.addGlyph(n, v, 0, 0) : 0;
@@ -428,14 +428,14 @@ export const SvgMo = (function <N, T, D>(): SvgMoClass<N, T, D> {
       x: number = 0
     ) {
       if (!n) return;
-      R = Math.max(0, R - HFUZZ); // A little less than the width of the right glyph
-      L = Math.max(0, L - HFUZZ); // A little less than the width of the left glyph
+      R = Math.max(0, R - HFUZZ); //           A little less than the width of the right glyph
+      L = Math.max(0, L - HFUZZ); //           A little less than the width of the left glyph
       const adaptor = this.adaptor;
       const [h, d, w] = this.getChar(n, v);
-      const X = W - L - R; // The width of the extender
-      const Y = h + d + 2 * VFUZZ; // The height (plus some fuzz) of the extender
-      const s = 1.5 * (X / w); // Scale the width so that left- and right-bearing won't hurt us
-      const D = -(d + VFUZZ); // The bottom position of the glyph
+      const X = W - L - R; //                 The width of the extender
+      const Y = h + d + 2 * VFUZZ; //         The height (plus some fuzz) of the extender
+      const s = 1.5 * (X / w); //             Scale the width so that left- and right-bearing won't hurt us
+      const D = -(d + VFUZZ); //              The bottom position of the glyph
       if (X <= 0) return;
       const svg = this.svg('svg', {
         width: this.fixed(X),
@@ -463,7 +463,7 @@ export const SvgMo = (function <N, T, D>(): SvgMoClass<N, T, D> {
      * @param {number} n   The character number for the right glyph of the stretchy character
      * @param {string} v   The variant for the right glyph
      * @param {number} W   The width of the stretched character
-     * @returns {number}    The width of the right glyph
+     * @returns {number}   The width of the right glyph
      */
     protected addRight(n: number, v: string, W: number): number {
       if (!n) return 0;