From 9255996cc2524bb0cc3d97ce7edb06721f42e8c5 Mon Sep 17 00:00:00 2001
From: Michael Kauzmann <michael.kauzmann@colorado.edu>
Date: Tue, 2 May 2023 11:57:04 -0600
Subject: [PATCH] don't propagate TODOs into generated files,
 https://github.com/phetsims/chipper/issues/740
 https://github.com/phetsims/chipper/issues/916

---
 quadrilateral_a11y_view.html | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/quadrilateral_a11y_view.html b/quadrilateral_a11y_view.html
index c09f915..b70b643 100644
--- a/quadrilateral_a11y_view.html
+++ b/quadrilateral_a11y_view.html
@@ -305,7 +305,6 @@ <h3>State Descriptions for Quadrilateral</h3>
       element.tabIndex = '-1';
 
       // make sure that styling is removed, unless some styling was added just for the copy
-      // TODO: pdom-style is not used anywhere. Should we keep it? https://github.com/phetsims/chipper/issues/916
       if ( element.className !== 'pdom-style' ) {
         element.removeAttribute( 'style' );
       }
@@ -382,7 +381,6 @@ <h3>State Descriptions for Quadrilateral</h3>
 
     // This is extremely inefficient every time the document changes, make a new copy, remove
     // the visual DOM, and add a new one
-    // TODO: Work on refining this, and only modifying the elements that change in the PDOM https://github.com/phetsims/chipper/issues/916
     copyContainer.innerHTML = '';
     const pdomCopy = pdomRoot.cloneNode( true );
     pdomCopy.removeAttribute( 'style' );
@@ -535,14 +533,12 @@ <h3>State Descriptions for Quadrilateral</h3>
 
       // strip the styling from the copied DOM elements
 
-      // TODO: why do we do this? Isn't this an element purely in the a11y view? https://github.com/phetsims/chipper/issues/916
       alertList.removeAttribute( 'style' );
 
       // get the parent container for the parallel DOM copy and the alert content
       const copyContainer = document.getElementById( 'dom-copy-container' );
 
       // once for initial setup
-      // TODO: do we have to do this? https://github.com/phetsims/chipper/issues/916
       setPDOMCopyContent( PDOMRoot, copyContainer );
 
       // update the PDOM copy whenever the sim's PDOM changes
@@ -561,7 +557,6 @@ <h3>State Descriptions for Quadrilateral</h3>
       document.getElementById( 'iframe' ).focus();
 
       // Provide fake focus highlighting to the PDOM copy based on what is actually highlighted in the sim.
-      // TODO: is polling the best way to do this? If so why 500? https://github.com/phetsims/chipper/issues/916
       let previousElement = null;
       setInterval( () => {