Merge pull request #1306 from drgrice1/update-npm-dependencies

Update third party npm dependencies.

Author: dlglin

htdocs/generate-assets.js

@@ -2,7 +2,8 @@
 
 /* eslint-env node */
 
-const yargs = require('yargs');
+const yargs = require('yargs/yargs');
+const { hideBin } = require('yargs/helpers');
 const chokidar = require('chokidar');
 const path = require('path');
 const { minify } = require('terser');
@@ -14,7 +15,7 @@ const postcss = require('postcss');
 const rtlcss = require('rtlcss');
 const cssMinify = require('cssnano');
 
-const argv = yargs
+const argv = yargs(hideBin(process.argv))
 	.usage('$0 Options')
 	.version(false)
 	.alias('help', 'h')

htdocs/js/GraphTool/graphtool.js

@@ -198,7 +198,7 @@ window.graphTool = (containerId, options) => {
 			}
 
 			if (this.visProp.useunicodeminus) labelText = labelText.replace(/-/g, '\u2212');
-			return addTeXDelims ?? this.visProp.label.usemathjax ? `\\(${labelText}\\)` : labelText;
+			return (addTeXDelims ?? this.visProp.label.usemathjax) ? `\\(${labelText}\\)` : labelText;
 		};
 
 		gt.board.defaultAxes.x.defaultTicks.generateLabelText = generateLabelText;

htdocs/js/MathQuill/mqeditor.js

@@ -550,7 +550,15 @@
 					toolbarEnabled = !toolbarEnabled;
 					localStorage.setItem('MQEditorToolbarEnabled', toolbarEnabled);
 					if (!toolbarEnabled && answerQuill.toolbar) toolbarRemove();
-					menu.hide();
+					// Bootstrap tries to focus the triggering element after hiding the menu. However, the menu gets
+					// disposed of and the hidden link which is the triggering element removed too quickly in the
+					// hidden.bs.dropdown event, and that causes an exception. So ignore that exception so that the
+					// answerQuill textarea is focused instead.
+					try {
+						menu.hide();
+					} catch {
+						/* ignore */
+					}
 					answerQuill.textarea.focus();
 				},
 				{ once: true }