Cleanup comment

Author: cpsievert

js/src/output.ts

@@ -27,17 +27,19 @@ class OutputManager extends HTMLManager {
 // Define our own custom module loader for Shiny
 const shinyRequireLoader = async function(moduleName: string, moduleVersion: string): Promise<any> {
 
-  // shiny provides a shim of require.js and also sets `define.amd=false` to
-  // allow <script>s with anonymous define()s to be loaded without error.
-  // Note that, when anonymous define()s are used, the shim will leverage the
-  // data-requiremodule attribute to determine the module name.
+  // shiny provides a shim of require.js which allows <script>s with anonymous
+  // define()s to be loaded without error. When an anonymous define() occurs,
+  // the shim uses the data-requiremodule attribute (set by require.js) on the script
+  // to determine the module name.
+  // https://github.com/posit-dev/py-shiny/blob/230940c/scripts/define-shims.js#L10-L16
   // In the context of shinywidgets, when a widget gets rendered, it should
-  // include another <script> tag that does `require.config({paths: {...}})`
-  // to map the module name to the URL of the widget's JS file.
+  // come with another <script> tag that does `require.config({paths: {...}})`
+  // which maps the module name to a URL of the widget's JS file.
   const oldAmd = (window as any).define.amd;
 
   // This is probably not necessary, but just in case -- especially now in a
   // anywidget/ES6 world, we probably don't want to load AMD modules
+  // (plotly is one example of a widget that will fail to load if AMD is enabled)
   (window as any).define.amd = false;
 
   // Store jQuery global since loading we load a module, it may overwrite it