@instructure/canvas-rce 5.11.1 → 5.12.2

package/CHANGELOG.md

@@ -5,6 +5,38 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## 5.12.2 - 2024-01-31
+
+### Changed
+
+- Moved RCE's makeAllExternalLinksExternalLinks
+- Removed doc-previews package
+- Removed CommonJS build of RCE
+
+## 5.12.1 - 2024-01-26
+
+### Fixed
+
+- An issue where we were adding file verifiers unnecessarily to non-user files
+  which were allowing students access to course files they should not have
+  access to
+- An issue where type query parameter was duplicated.
+- Fix focus ring in RCE content
+- Stop adding wrap params to course links
+
+### Changed
+
+- Bump redux-thunk to 3.1.0
+- Show full name in hover in All Files tray
+- Remove "scroll-into-view"
+- jQuery changes
+- Upgrade moment to 0.5.43
+- Upgrade babel-loader to 9.1.3
+- Remove use of InferType
+- Show media captions in New Quizzes
+- Bump Instui to 8.49
+
+
 ## 5.11.1 - 2023-10-12
 
 ### Fixed

package/babel-register.js

@@ -22,13 +22,6 @@ require('@babel/register')({
   presets: [['@babel/preset-env'], ['@babel/preset-react', {}]],
   plugins: [
     ['@babel/plugin-proposal-decorators', {legacy: true}],
-    [
-      '@instructure/babel-plugin-themeable-styles',
-      {
-        postcssrc: require('@instructure/ui-postcss-config')()(),
-        themeablerc: {},
-      },
-    ],
   ],
   targets: {
     node: 'current',

package/babel.config.cjs.js

@@ -40,12 +40,5 @@ module.exports = {
       },
     ],
     ['@babel/plugin-proposal-decorators', {legacy: true}],
-    [
-      '@instructure/babel-plugin-themeable-styles',
-      {
-        postcssrc: require('@instructure/ui-postcss-config')()(),
-        themeablerc: {},
-      },
-    ],
   ],
 }

package/babel.config.js

@@ -84,15 +84,6 @@ module.exports = {
     ],
 
     ['@babel/plugin-proposal-decorators', {legacy: true}],
-
-    [
-      '@instructure/babel-plugin-themeable-styles',
-      {
-        ignore: () => false,
-        postcssrc: require('@instructure/ui-postcss-config')()(),
-        themeablerc: {},
-      },
-    ],
   ],
 
   targets: {

package/es/canvasFileBrowser/FileBrowser.js

@@ -340,9 +340,14 @@ class FileBrowser extends React.Component {
       itemIcon = getIconFromType(file.type);
     }
 
+    const containerRef = node => {
+      if (node && !node.title && props.name) node.title = props.name;
+    };
+
     return { ...props,
       thumbnail,
-      itemIcon
+      itemIcon,
+      containerRef
     };
   }
 

package/es/common/fileUrl.js

@@ -104,9 +104,11 @@ export function fixupFileUrl(contextType, contextId, fileInfo, canvasOrigin) {
     parsed = changeDownloadToWrapParams(parsed);
     parsed = addContext(parsed, contextType, contextId); // if this is a user file, add the verifier
 
-    if (fileInfo.uuid && (contextType.includes('user') || !!canvasOrigin && canvasOrigin !== window.location.origin)) {
+    if (fileInfo.uuid && contextType.includes('user')) {
       delete parsed.search;
       parsed.query.verifier = fileInfo.uuid;
+    } else {
+      delete parsed.query.verifier;
     }
 
     fileInfo[key] = format(parsed);
@@ -136,8 +138,6 @@ export function prepEmbedSrc(url) {
   return format(parsed);
 } // when the user opens a link to a resource, we want its view
 // logged, so remove /preview
-// Add wrap=1 to indicate clicking on the link should open a preview
-// and not download the file (this doesn't work if the original link is a download link)
 
 export function prepLinkedSrc(url) {
   const parsed = parseCanvasUrl(url);
@@ -148,6 +148,5 @@ export function prepLinkedSrc(url) {
 
   delete parsed.search;
   parsed.pathname = parsed.pathname.replace(/\/preview(\?|$)/, '$1');
-  parsed.query.wrap = '1';
   return format(parsed);
 }

package/es/defaultTinymceConfig.js

@@ -71,7 +71,7 @@ const defaultTinymceConfig = {
   // then edited.
   //
   // this list needs to be kept in sync with the list in gems/canvas_sanitize/lib/canvas_sanitize/canvas_sanitize.rb
-  valid_elements: '@[id|class|style|title|dir<ltr?rtl|lang|xml::lang|role],a[rel|rev|charset|hreflang|tabindex|accesskey|type|name|href|target|title|class],strong/b,em/i,strike/s,u,#p,-ol[type|compact],-ul[type|compact],-li,br,img[longdesc|usemap|src|border|alt|title|hspace|vspace|width|height|align|role],-sub,-sup,-blockquote[cite],-table[border=0|cellspacing|cellpadding|width|frame|rules|height|align|summary|bgcolor|background|bordercolor],-tr[rowspan|width|height|align|valign|bgcolor|background|bordercolor],tbody,thead,tfoot,#td[colspan|rowspan|width|height|align|valign|bgcolor|background|bordercolor|scope],#th[colspan|rowspan|width|height|align|valign|scope],caption,-div,-span,-code,-pre,address,-h1,-h2,-h3,-h4,-h5,-h6,hr[size|noshade],-font[face|size|color],dd,dl,dt,cite,abbr,acronym,del[datetime|cite],ins[datetime|cite],object[classid|width|height|codebase|*],param[name|value|_value],embed[type|width|height|src|*],map[name],area[shape|coords|href|alt|target],bdo,col[align|char|charoff|span|valign|width],colgroup[align|char|charoff|span|valign|width],dfn,kbd,q[cite],samp,small,tt,var,big,figure,figcaption,source[media|sizes|src|srcset|type],track,mark,article,aside,details,footer,header,nav,section,summary,time',
+  valid_elements: '@[id|class|style|title|dir<ltr?rtl|lang|xml::lang|role],a[rel|rev|charset|hreflang|tabindex|accesskey|type|name|href|target|title|class],strong/b,em/i,strike/s,u,#p,-ol[type|compact],-ul[type|compact],-li,br,img[longdesc|usemap|src|border|alt|title|hspace|vspace|width|height|align|role],-sub,-sup,-blockquote[cite],-table[border=0|cellspacing|cellpadding|width|frame|rules|height|align|summary|bgcolor|background|bordercolor],-tr[rowspan|width|height|align|valign|bgcolor|background|bordercolor],tbody,thead,tfoot,#td[colspan|rowspan|width|height|align|valign|bgcolor|background|bordercolor|scope],#th[colspan|rowspan|width|height|align|valign|scope],caption,-div,-span,-code,-pre,address,-h1,-h2,-h3,-h4,-h5,-h6,hr[size|noshade],-font[face|size|color],dd,dl,dt,cite,abbr,acronym,del[datetime|cite],ins[datetime|cite],object[classid|width|height|codebase|*],param[name|value|_value],embed[type|width|height|src|*],map[name],area[shape|coords|href|alt|target],bdo,col[align|char|charoff|span|valign|width],colgroup[align|char|charoff|span|valign|width],dfn,kbd,q[cite],samp,small,tt,var,big,figure,figcaption,source[media|width|height|sizes|src|srcset|type],track,mark,article,aside,details,footer,header,nav,section,summary,time',
   extended_valid_elements: '@[id|accesskey|class|dir|lang|style|tabindex|title|contenteditable|contextmenu|draggable|dropzone|hidden|longdesc|spellcheck|translate|align|role|aria-labelledby|aria-atomic|aria-busy|aria-controls|aria-describedby|aria-disabled|aria-dropeffect|aria-flowto|aria-grabbed|aria-haspopup|aria-hidden|aria-invalid|aria-label|aria-labelledby|aria-live|aria-owns|aria-relevant|aria-autocomplete|aria-checked|aria-disabled|aria-expanded|aria-haspopup|aria-hidden|aria-invalid|aria-label|aria-level|aria-multiline|aria-multiselectable|aria-orientation|aria-pressed|aria-readonly|aria-required|aria-selected|aria-sort|aria-valuemax|aria-valuemin|aria-valuenow|aria-valuetext],iframe[id|data-media-type|title|src|width|height|name|align|style|class|sandbox|allowfullscreen|webkitallowfullscreen|mozallowfullscreen|allow],i[iclass],a[hidden|href|target|rel|media|hreflang|type|charset|name|rev|shape|coords|download|alt],#p,li[value],-ol[reversed|start|type|compact],pre[width],table[border|summary|width|frame|rules|cellspacing|cellpadding|bgcolor],tbody[char|charoff|valign],td[colspan|rowspan|headers|abbr|axis|scope|align|char|charoff|valign|nowrap|bgcolor|width|height],tfoot[char|charoff|valign],th[colspan|rowspan|headers|scope|abbr|axis|align|char|charoff|valign|nowrap|bgcolor|width|height],thead[char|charoff|valign],tr[char|charoff|valign|bgcolor],-ul[compact],video[name|src|allowfullscreen|muted|poster|width|height|controls|playsinline],audio[name|src|muted|controls],annotation[href|xref|definitionURL|encoding|cd|name|src],annotation-xml[href|xref|definitionURL|encoding|cd|name|src],maction[href|xref|mathcolor|mathbackground|actiontype|selection],maligngroup[href|xref|mathcolor|mathbackground|groupalign],malignmark[href|xref|mathcolor|mathbackground|edge],math[xmlns|href|xref|display|maxwidth|overflow|altimg|altimg-width|altimg-height|altimg-valign|alttext|cdgroup|mathcolor|mathbackground|scriptlevel|displaystyle|scriptsizemultiplier|scriptminsize|infixlinebreakstyle|decimalpoint|mathvariant|mathsize|width|height|valign|form|fence|separator|lspace|rspace|stretchy|symmetric|maxsize|minsize|largeop|movablelimits|accent|linebreak|lineleading|linebreakstyle|linebreakmultchar|indentalign|indentshift|indenttarget|indentalignfirst|indentshiftfirst|indentalignlast|indentshiftlast|depth|lquote|rquote|linethickness|munalign|denomalign|bevelled|voffset|open|close|separators|notation|subscriptshift|superscriptshift|accentunder|align|rowalign|columnalign|groupalign|alignmentscope|columnwidth|rowspacing|columnspacing|rowlines|columnlines|frame|framespacing|equalrows|equalcolumns|side|minlabelspacing|rowspan|columnspan|edge|stackalign|charalign|charspacing|longdivstyle|position|shift|location|crossout|length|leftoverhang|rightoverhang|mslinethickness|selection],menclose[href|xref|mathcolor|mathbackground|notation],merror[href|xref|mathcolor|mathbackground],mfenced[href|xref|mathcolor|mathbackground|open|close|separators],mfrac[href|xref|mathcolor|mathbackground|linethickness|munalign|denomalign|bevelled],mglyph[href|xref|mathcolor|mathbackground|src|alt|width|height|valign],mi[href|xref|mathcolor|mathbackground|mathvariant|mathsize],mlabeledtr[href|xref|mathcolor|mathbackground],mlongdiv[href|xref|mathcolor|mathbackground|longdivstyle|align|stackalign|charalign|charspacing],mmultiscripts[href|xref|mathcolor|mathbackground|subscriptshift|superscriptshift],mn[href|xref|mathcolor|mathbackground|mathvariant|mathsize],mo[href|xref|mathcolor|mathbackground|mathvariant|mathsize|form|fence|separator|lspace|rspace|stretchy|symmetric|maxsize|minsize|largeop|movablelimits|accent|linebreak|lineleading|linebreakstyle|linebreakmultchar|indentalign|indentshift|indenttarget|indentalignfirst|indentshiftfirst|indentalignlast|indentshiftlast],mover[href|xref|mathcolor|mathbackground|accent|align],mpadded[href|xref|mathcolor|mathbackground|height|depth|width|lspace|voffset],mphantom[href|xref|mathcolor|mathbackground],mprescripts[href|xref|mathcolor|mathbackground],mroot[href|xref|mathcolor|mathbackground],mrow[href|xref|mathcolor|mathbackground],ms[href|xref|mathcolor|mathbackground|mathvariant|mathsize|lquote|rquote],mscarries[href|xref|mathcolor|mathbackground|position|location|crossout|scriptsizemultiplier],mscarry[href|xref|mathcolor|mathbackground|location|crossout],msgroup[href|xref|mathcolor|mathbackground|position|shift],msline[href|xref|mathcolor|mathbackground|position|length|leftoverhang|rightoverhang|mslinethickness],mspace[href|xref|mathcolor|mathbackground|mathvariant|mathsize],msqrt[href|xref|mathcolor|mathbackground],msrow[href|xref|mathcolor|mathbackground|position],mstack[href|xref|mathcolor|mathbackground|align|stackalign|charalign|charspacing],mstyle[href|xref|mathcolor|mathbackground|scriptlevel|displaystyle|scriptsizemultiplier|scriptminsize|infixlinebreakstyle|decimalpoint|mathvariant|mathsize|width|height|valign|form|fence|separator|lspace|rspace|stretchy|symmetric|maxsize|minsize|largeop|movablelimits|accent|linebreak|lineleading|linebreakstyle|linebreakmultchar|indentalign|indentshift|indenttarget|indentalignfirst|indentshiftfirst|indentalignlast|indentshiftlast|depth|lquote|rquote|linethickness|munalign|denomalign|bevelled|voffset|open|close|separators|notation|subscriptshift|superscriptshift|accentunder|align|rowalign|columnalign|groupalign|alignmentscope|columnwidth|rowspacing|columnspacing|rowlines|columnlines|frame|framespacing|equalrows|equalcolumns|side|minlabelspacing|rowspan|columnspan|edge|stackalign|charalign|charspacing|longdivstyle|position|shift|location|crossout|length|leftoverhang|rightoverhang|mslinethickness|selection],msub[href|xref|mathcolor|mathbackground|subscriptshift],msubsup[href|xref|mathcolor|mathbackground|subscriptshift|superscriptshift],msup[href|xref|mathcolor|mathbackground|superscriptshift],mtable[href|xref|mathcolor|mathbackground|align|rowalign|columnalign|groupalign|alignmentscope|columnwidth|width|rowspacing|columnspacing|rowlines|columnlines|frame|framespacing|equalrows|equalcolumns|displaystyle|side|minlabelspacing],mtd[href|xref|mathcolor|mathbackground|rowspan|columnspan|rowalign|columnalign|groupalign],mtext[href|xref|mathcolor|mathbackground|mathvariant|mathsize|width|height|depth|linebreak],mtr[href|xref|mathcolor|mathbackground|rowalign|columnalign|groupalign],munder[href|xref|mathcolor|mathbackground|accentunder|align],munderover[href|xref|mathcolor|mathbackground|accent|accentunder|align],none[href|xref|mathcolor|mathbackground],semantics[href|xref|definitionURL|encoding],picture,ruby,rp,rt,wbr' + // the svg necessary for the uploading placeholder's spinner
   'svg[*],g[*],circle[*]',
   non_empty_elements: 'td th iframe video audio object script a i area base basefont br col frame hr img input isindex link meta param embed source wbr track ruby',

package/{lib/rce/plugins/instructure_rce_external_tools/lti13-content-items/Lti13ContentItemJson.js → es/elementDenylist.js}

@@ -15,9 +15,5 @@
  * You should have received a copy of the GNU Affero General Public License along
  * with this program. If not, see <http://www.gnu.org/licenses/>.
  */
-export function isContentItemThumbnailJson(input) {
-  return typeof input === 'object' && typeof input.url === 'string';
-}
-export function isContentItemIframeJson(input) {
-  return typeof input === 'object' && typeof input.src === 'string';
-}
+const elementDenylist = ('html,base,head,link,meta,style,title,body,main,data,rb,rtc,portal,svg,canvas,noscript,script,button,datalist,fieldset,form,input,label,legend,meter,optgroup' + ',option,output,progress,select,textarea,dialog,menu,slot,template,applet,basefont,bgsound,blink,center,content,dir,frame,frameset,hgroup,image,keygen,marquee,menuitem,nobr,noembed' + ',noframes,plaintext,rb,rtc,shadow,spacer,xmp').split(',');
+export default elementDenylist;

package/es/enhance-user-content/doc_previews.js

@@ -92,6 +92,10 @@ export function loadDocPreview($container, options) {
   };
   if (!$container) return; // an issue in specs
 
+  if (!($container instanceof HTMLElement)) {
+    throw new Error('loadDocPreview requires a DOM element as first argument');
+  }
+
   function tellAppIViewedThisInline() {
     // if I have a url to ping back to the app that I viewed this file inline, ping it.
     if (opts.attachment_view_inline_ping_url) {

package/es/enhance-user-content/enhance_user_content.js

@@ -15,16 +15,16 @@
  * You should have received a copy of the GNU Affero General Public License along
  * with this program. If not, see <http://www.gnu.org/licenses/>.
  */
-import { IconDownloadLine, IconExternalLinkLine } from '@instructure/ui-icons/es/svg';
+import { IconDownloadLine } from '@instructure/ui-icons/es/svg';
 import formatMessage from '../format-message';
 import { closest, getData, hide, insertAfter, setData, show } from './jqueryish_funcs';
-import { getTld, isExternalLink, showFilePreview, youTubeID } from './instructure_helper';
+import { isExternalLink, showFilePreview, youTubeID } from './instructure_helper';
 import mediaCommentThumbnail from './media_comment_thumbnail';
 import { addParentFrameContextToUrl } from '../rce/plugins/instructure_rce_external_tools/util/addParentFrameContextToUrl';
-import Mathml, { MathJaxDirective } from './mathml'; // in jest the es directory doesn't exist so stub the undefined svg
+import { MathJaxDirective, Mathml } from './mathml';
+import { makeExternalLinkIcon } from './external_links'; // in jest the es directory doesn't exist so stub the undefined svg
 
 const IconDownloadSVG = (IconDownloadLine === null || IconDownloadLine === void 0 ? void 0 : IconDownloadLine.src) || '<svg></svg>';
-const IconExternalLinkSVG = (IconExternalLinkLine === null || IconExternalLinkLine === void 0 ? void 0 : IconExternalLinkLine.src) || '<svg></svg>';
 
 function makeDownloadButton(download_url, filename) {
   const a = document.createElement('a');
@@ -47,22 +47,6 @@ function makeDownloadButton(download_url, filename) {
   return a;
 }
 
-function makeExternalLinkIcon(forLink) {
-  const dir = forLink && window.getComputedStyle(forLink).direction || 'ltr';
-  const $icon = document.createElement('span');
-  $icon.setAttribute('class', 'external_link_icon');
-  const style = `margin-inline-start: 5px; display: inline-block; text-indent: initial; ${dir === 'rtl' ? 'transform:scale(-1, 1)' : ''}`;
-  $icon.setAttribute('style', style);
-  $icon.setAttribute('role', 'presentation');
-  $icon.innerHTML = IconExternalLinkSVG;
-  $icon.firstChild.setAttribute('style', 'width:1em; height:1em; vertical-align:middle; fill:currentColor');
-  const srspan = document.createElement('span');
-  srspan.setAttribute('class', 'screenreader-only');
-  srspan.textContent = formatMessage('Links to an external site.');
-  $icon.appendChild(srspan);
-  return $icon;
-}
-
 function handleYoutubeLink($link) {
   const href = $link.getAttribute('href');
   const id = youTubeID(href || '');
@@ -354,43 +338,4 @@ export function enhanceUserContent() {
     const src = frame.src;
     frame.src = src;
   });
-}
-export function makeAllExternalLinksExternalLinks() {
-  // in 100ms (to give time for everything else to load), find all the external links and add give them
-  // the external link look and behavior (force them to open in a new tab)
-  setTimeout(function () {
-    const content = document.getElementById('content');
-    if (!content) return;
-    const tld = getTld(window.location.hostname);
-    const links = content.querySelectorAll(`a[href*="//"]:not([href*="${tld}"])`); // technique for finding "external" links copied from https://davidwalsh.name/external-links-css
-
-    for (let i = 0; i < links.length; i++) {
-      const $link = links[i]; // don't mess with the ones that were already processed in enhanceUserContent
-
-      if ($link.classList.contains('external')) continue;
-      if ($link.matches('.open_in_a_new_tab')) continue;
-      if ($link.querySelectorAll('img').length > 0) continue;
-      if ($link.matches('.not_external')) continue;
-      if ($link.matches('.exclude_external_icon')) continue; // we have some pre-instui buttons that are styled links
-
-      if ($link.classList.contains('btn')) continue;
-      const $linkToReplace = $link;
-
-      if ($linkToReplace) {
-        const $linkIndicator = makeExternalLinkIcon();
-        $linkToReplace.classList.add('external');
-        $linkToReplace.querySelectorAll('span.ui-icon-extlink').forEach(c => c.remove);
-        $linkToReplace.setAttribute('target', '_blank');
-        $linkToReplace.setAttribute('rel', 'noreferrer noopener');
-        const $linkSpan = document.createElement('span');
-        const $linkText = $linkToReplace.innerHTML;
-        $linkSpan.innerHTML = $linkText;
-
-        while ($linkToReplace.firstChild) $linkToReplace.removeChild($linkToReplace.firstChild);
-
-        $linkToReplace.appendChild($linkSpan);
-        $linkToReplace.appendChild($linkIndicator);
-      }
-    }
-  }, 100);
 }

package/es/enhance-user-content/external_links.js

@@ -0,0 +1,75 @@
+/*
+ * Copyright (C) 2011 - present Instructure, Inc.
+ *
+ * This file is part of Canvas.
+ *
+ * Canvas is free software: you can redistribute it and/or modify it under
+ * the terms of the GNU Affero General Public License as published by the Free
+ * Software Foundation, version 3 of the License.
+ *
+ * Canvas is distributed in the hope that it will be useful, but WITHOUT ANY
+ * WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
+ * A PARTICULAR PURPOSE. See the GNU Affero General Public License for more
+ * details.
+ *
+ * You should have received a copy of the GNU Affero General Public License along
+ * with this program. If not, see <http://www.gnu.org/licenses/>.
+ */
+import { IconExternalLinkLine } from '@instructure/ui-icons/es/svg';
+import { getTld } from './instructure_helper';
+import formatMessage from '../format-message';
+const IconExternalLinkSVG = (IconExternalLinkLine === null || IconExternalLinkLine === void 0 ? void 0 : IconExternalLinkLine.src) || '<svg></svg>';
+export function makeExternalLinkIcon(forLink) {
+  const dir = forLink && window.getComputedStyle(forLink).direction || 'ltr';
+  const $icon = document.createElement('span');
+  $icon.setAttribute('class', 'external_link_icon');
+  const style = `margin-inline-start: 5px; display: inline-block; text-indent: initial; ${dir === 'rtl' ? 'transform:scale(-1, 1)' : ''}`;
+  $icon.setAttribute('style', style);
+  $icon.setAttribute('role', 'presentation');
+  $icon.innerHTML = IconExternalLinkSVG;
+  $icon.firstChild.setAttribute('style', 'width:1em; height:1em; vertical-align:middle; fill:currentColor');
+  const srspan = document.createElement('span');
+  srspan.setAttribute('class', 'screenreader-only');
+  srspan.textContent = formatMessage('Links to an external site.');
+  $icon.appendChild(srspan);
+  return $icon;
+}
+export function makeAllExternalLinksExternalLinks() {
+  // in 100ms (to give time for everything else to load), find all the external links and add give them
+  // the external link look and behavior (force them to open in a new tab)
+  setTimeout(function () {
+    const content = document.getElementById('content');
+    if (!content) return;
+    const tld = getTld(window.location.hostname);
+    const links = content.querySelectorAll(`a[href*="//"]:not([href*="${tld}"])`); // technique for finding "external" links copied from https://davidwalsh.name/external-links-css
+
+    for (let i = 0; i < links.length; i++) {
+      const $link = links[i]; // don't mess with the ones that were already processed in enhanceUserContent
+
+      if ($link.classList.contains('external')) continue;
+      if ($link.matches('.open_in_a_new_tab')) continue;
+      if ($link.querySelectorAll('img').length > 0) continue;
+      if ($link.matches('.not_external')) continue;
+      if ($link.matches('.exclude_external_icon')) continue; // we have some pre-instui buttons that are styled links
+
+      if ($link.classList.contains('btn')) continue;
+      const $linkToReplace = $link;
+
+      if ($linkToReplace) {
+        const $linkIndicator = makeExternalLinkIcon();
+        $linkToReplace.classList.add('external');
+        $linkToReplace.querySelectorAll('span.ui-icon-extlink').forEach(c => c.remove);
+        $linkToReplace.setAttribute('target', '_blank');
+        $linkToReplace.setAttribute('rel', 'noreferrer noopener');
+        const $linkSpan = document.createElement('span');
+        const $linkText = $linkToReplace.innerHTML;
+        $linkSpan.innerHTML = $linkText;
+
+        while ($linkToReplace.firstChild) $linkToReplace.removeChild($linkToReplace.firstChild);
+
+        $linkToReplace.appendChild($linkSpan);
+        $linkToReplace.appendChild($linkIndicator);
+      }
+    }
+  }, 100);
+}

package/es/enhance-user-content/mathml.js

@@ -15,7 +15,6 @@
  * You should have received a copy of the GNU Affero General Public License along
  * with this program. If not, see <http://www.gnu.org/licenses/>.
  */
-import $ from 'jquery';
 const MathJaxDirective = Object.freeze({
   Ignore: 'mathjax_ignore',
   Process: 'mathjax_process'
@@ -130,9 +129,9 @@ class Mathml {
             elem.querySelectorAll('.math_equation_latex').forEach(m => m.classList.add('fade-in-equation'));
           });
         } else {
-          // though isMathInElement ignores <math> w/in .hidden-readable elements,
+          // Though isMathInElement ignores <math> with/in .hidden-readable elements,
           // MathJax does not and will process it anyway. This is a problem when
-          // you open the equation editor while editing a quiz and it adds to the DOM
+          // you open the equation editor while editing a quiz, and it adds to the DOM
           // elements that will get saved with the quiz.
           // There's a feature request against MathJax (https://github.com/mathjax/MathJax/issues/505)
           // to add an ignoreClass config prop to the mml2jax processor, but it's not available.
@@ -140,7 +139,10 @@ class Mathml {
           // right after MathJax adds it.
           window.MathJax.Hub.Register.MessageHook('End Math', function (message) {
             const elm = Array.isArray(message[1]) ? message[1][0] : message[1];
-            $(elm).find('.hidden-readable [class^="MathJax"], .hidden-readable [id^="MathJax"]').remove();
+            const elements = elm.querySelectorAll('.hidden-readable [class^="MathJax"], .hidden-readable [id^="MathJax"]');
+            elements.forEach(function (element) {
+              element.remove();
+            });
           });
         } // leaving this here so I don't have to keep looking up how to see all messages
         // window.MathJax.Hub.Startup.signal.Interest(function (message) {
@@ -198,9 +200,9 @@ class Mathml {
     const mathElements = elem.getElementsByTagName('math');
 
     for (let i = 0; i < mathElements.length; i++) {
-      const $el = $(mathElements[i]);
+      const el = mathElements[i];
 
-      if ($el.is(':visible') && $el.parent('.hidden-readable').length <= 0 && $el.parent('.MJX_Assistive_MathML').length <= 0 // already mathjax'd
+      if (el.offsetParent !== null && !el.closest('.hidden-readable') && !el.closest('.MJX_Assistive_MathML') // already mathjax'd
       ) {
         return true;
       }
@@ -394,4 +396,4 @@ function handleNewMath(event) {
 }
 
 window.addEventListener(Mathml.processNewMathEventName, handleNewMath);
-export { Mathml as default, mathImageHelper, MathJaxDirective };
+export { Mathml, mathImageHelper, MathJaxDirective };

package/es/getThemeVars.js

@@ -24,7 +24,7 @@ let memoizedVariables;
 function getThemeVars() {
   if (memoizedVariables) return memoizedVariables;
   const {
-    defaultThemeKey,
+    currentThemeKey,
     overrides,
     themes
   } = getRegistry(); // Just assume the "canvas" theme if the default key is null. This will
@@ -33,11 +33,12 @@ function getThemeVars() {
   // Also we have to cloneDeep this because the merge below is about to
   // mutate the whole thing.
 
-  const variables = cloneDeep(themes[defaultThemeKey !== null && defaultThemeKey !== void 0 ? defaultThemeKey : 'canvas']);
+  const themeKey = currentThemeKey || 'canvas';
+  const variables = cloneDeep(themes[themeKey]);
   merge(variables, overrides);
   memoizedVariables = {
     variables,
-    key: defaultThemeKey
+    key: themeKey
   };
   return memoizedVariables;
 }

package/es/getTranslations.js

@@ -28,6 +28,10 @@ export default function getTranslations(locale) {
       let p;
 
       switch (locale) {
+        case 'ab':
+          p = import('./translations/locales/ab');
+          break;
+
         case 'ar':
           p = import('./translations/locales/ar');
           break;
@@ -36,6 +40,14 @@ export default function getTranslations(locale) {
           p = import('./translations/locales/ca');
           break;
 
+        case 'cs':
+          p = import('./translations/locales/cs');
+          break;
+
+        case 'cs-CZ':
+          p = import('./translations/locales/cs_CZ');
+          break;
+
         case 'cy':
           p = import('./translations/locales/cy');
           break;
@@ -48,6 +60,10 @@ export default function getTranslations(locale) {
           p = import('./translations/locales/da-x-k12');
           break;
 
+        case 'da-DK':
+          p = import('./translations/locales/da_DK');
+          break;
+
         case 'de':
           p = import('./translations/locales/de');
           break;
@@ -84,6 +100,18 @@ export default function getTranslations(locale) {
           p = import('./translations/locales/en_GB');
           break;
 
+        case 'en-NZ':
+          p = import('./translations/locales/en_NZ');
+          break;
+
+        case 'en-SE':
+          p = import('./translations/locales/en_SE');
+          break;
+
+        case 'en-US':
+          p = import('./translations/locales/en_US');
+          break;
+
         case 'es':
           p = import('./translations/locales/es');
           break;
@@ -124,10 +152,22 @@ export default function getTranslations(locale) {
           p = import('./translations/locales/hu');
           break;
 
+        case 'hu-HU':
+          p = import('./translations/locales/hu_HU');
+          break;
+
         case 'hy':
           p = import('./translations/locales/hy');
           break;
 
+        case 'id':
+          p = import('./translations/locales/id');
+          break;
+
+        case 'id-ID':
+          p = import('./translations/locales/id_ID');
+          break;
+
         case 'is':
           p = import('./translations/locales/is');
           break;
@@ -144,10 +184,26 @@ export default function getTranslations(locale) {
           p = import('./translations/locales/ko');
           break;
 
+        case 'ko-KR':
+          p = import('./translations/locales/ko_KR');
+          break;
+
+        case 'lt':
+          p = import('./translations/locales/lt');
+          break;
+
+        case 'lt-LT':
+          p = import('./translations/locales/lt_LT');
+          break;
+
         case 'mi':
           p = import('./translations/locales/mi');
           break;
 
+        case 'mn-MN':
+          p = import('./translations/locales/mn_MN');
+          break;
+
         case 'ms':
           p = import('./translations/locales/ms');
           break;
@@ -164,6 +220,10 @@ export default function getTranslations(locale) {
           p = import('./translations/locales/nl');
           break;
 
+        case 'nl-NL':
+          p = import('./translations/locales/nl_NL');
+          break;
+
         case 'nn':
           p = import('./translations/locales/nn');
           break;
@@ -180,6 +240,10 @@ export default function getTranslations(locale) {
           p = import('./translations/locales/pt_BR');
           break;
 
+        case 'ro':
+          p = import('./translations/locales/ro');
+          break;
+
         case 'ru':
           p = import('./translations/locales/ru');
           break;
@@ -216,6 +280,10 @@ export default function getTranslations(locale) {
           p = import('./translations/locales/th_TH');
           break;
 
+        case 'tl-PH':
+          p = import('./translations/locales/tl_PH');
+          break;
+
         case 'tr':
           p = import('./translations/locales/tr');
           break;
@@ -248,6 +316,14 @@ export default function getTranslations(locale) {
           p = import('./translations/locales/zh_HK');
           break;
 
+        case 'zh-TW':
+          p = import('./translations/locales/zh_TW');
+          break;
+
+        case 'zh-TW.Big5':
+          p = import('./translations/locales/zh_TW.Big5');
+          break;
+
         default:
           p = Promise.resolve(null);
       }
@@ -260,5 +336,5 @@ export default function getTranslations(locale) {
   return transReadyPromise;
 }
 export function getLocaleList() {
-  return ['ar', 'ca', 'cy', 'da', 'da-x-k12', 'de', 'el', 'en', 'en-AU-x-unimelb', 'en-GB-x-ukhe', 'en-AU', 'en-CA', 'en-CY', 'en-GB', 'es', 'es-ES', 'es-GT', 'fa-IR', 'fi', 'fr', 'fr-CA', 'he', 'ht', 'hu', 'hy', 'is', 'it', 'ja', 'ko', 'mi', 'ms', 'nb', 'nb-x-k12', 'nl', 'nn', 'pl', 'pt', 'pt-BR', 'ru', 'se', 'sl', 'sv', 'sv-x-k12', 'sv-SE', 'tg', 'th', 'th-TH', 'tr', 'uk-UA', 'vi', 'vi-VN', 'zh', 'zh-Hans', 'zh-Hant', 'zh-HK'];
+  return ['ab', 'ar', 'ca', 'cs', 'cs-CZ', 'cy', 'da', 'da-x-k12', 'da-DK', 'de', 'el', 'en', 'en-AU-x-unimelb', 'en-GB-x-ukhe', 'en-AU', 'en-CA', 'en-CY', 'en-GB', 'en-NZ', 'en-SE', 'en-US', 'es', 'es-ES', 'es-GT', 'fa-IR', 'fi', 'fr', 'fr-CA', 'he', 'ht', 'hu', 'hu-HU', 'hy', 'id', 'id-ID', 'is', 'it', 'ja', 'ko', 'ko-KR', 'lt', 'lt-LT', 'mi', 'mn-MN', 'ms', 'nb', 'nb-x-k12', 'nl', 'nl-NL', 'nn', 'pl', 'pt', 'pt-BR', 'ro', 'ru', 'se', 'sl', 'sv', 'sv-x-k12', 'sv-SE', 'tg', 'th', 'th-TH', 'tl-PH', 'tr', 'uk-UA', 'vi', 'vi-VN', 'zh', 'zh-Hans', 'zh-Hant', 'zh-HK', 'zh-TW', 'zh-TW.Big5'];
 }

package/es/index.js

@@ -21,7 +21,7 @@ import { headerFor, originFromHost } from './rcs/api';
 import getTranslations from './getTranslations';
 import defaultTinymceConfig from './defaultTinymceConfig';
 import { setLocale } from './common/natcompare';
-import Mathml from './enhance-user-content/mathml';
+import { Mathml } from './enhance-user-content/mathml';
 export * from './enhance-user-content/index';
 export const defaultConfiguration = defaultTinymceConfig;
 export { instuiPopupMountNode } from './util/fullscreenHelpers';