update

Author: SharonStrats

src/widgets/modals.ts

@@ -9,16 +9,16 @@ export type ListItem = {
   link: string
 }
 
-// Two functions that need to be implemented to use the modal
-// When the user clicks the button, open the modal
+// Two functions that need to be implemented to use the modal.
+// When the user clicks the button, open the modal.
 /* Click handler on the button to display it.
 btn.onclick = function() {
   modal.style.display = "block";
 }
 
-// When the user clicks anywhere outside of the modal, close it
-Window click handler so that the modal will close
-even if the user doesn't click close
+// When the user clicks anywhere outside of the modal, close it.
+// Window click handler so that the modal will close
+// even if the user doesn't click close.
 window.onclick = function(event) {
   if (event.target == modal) {
     modal.style.display = "none";
@@ -68,7 +68,7 @@ const createListItems = (dom: HTMLDocument, list: ListItem) => {
   const link: HTMLAnchorElement = dom.createElement('a')
   link.setAttribute('style', style.modalAnchorStyle)
   link.href = list.link
-  link.innerHTML = list.label
+  link.textContent = list.label
   li.appendChild(link)
   return li
 }

src/widgets/modalsStyle.ts

@@ -1,6 +1,6 @@
 /**
- * Get the button style, based on options.
- * See https://design.inrupt.com/atomic-core/?cat=Atoms#Buttons
+ * Get the modal styles, based on options such as position and background overlay.
+ * See https://design.inrupt.com/atomic-core/?cat=Organisms#Modals for modal design guidelines.
  */
 export type ModalWidgetStyleOptions = {
   topPosition?: string,