add support for dynamic layout on Chart Intro screen, phetsims/build-…

…a-nucleus#90
phetsims · Jul 31, 2023 · 251b9f5 · 251b9f5
1 parent 05001ae
commit 251b9f5
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/js/nodes/RichText.ts b/js/nodes/RichText.ts
@@ -108,8 +108,8 @@ export type RichTextHref = ( () => void ) | string;
 type RichTextLinksObject = Record<string, RichTextHref>;
 export type RichTextLinks = RichTextLinksObject | true;
 
-// Used only for guarding against assertions, we want to know that we aren't in dynamic stringTest mode
-const isNotDynamicStringTest = !_.hasIn( window, 'phet.chipper.queryParameters' ) || window.phet.chipper.queryParameters.stringTest !== 'dynamic';
+// Used only for guarding against assertions, we want to know that we aren't in stringTesting mode
+const isStringTest = window.QueryStringMachine && QueryStringMachine.containsKey( 'stringTest' );
 
 type SelfOptions = {
   // Sets how bounds are determined for text
@@ -582,12 +582,12 @@ export default class RichText extends Node {
     if ( assert ) {
       if ( this._links && this._links !== true ) {
         Object.keys( this._links ).forEach( link => {
-          assert && allowLinksProperty.value && isNotDynamicStringTest && assert( usedLinks.includes( link ), `Unused RichText link: ${link}` );
+          assert && allowLinksProperty.value && !isStringTest && assert( usedLinks.includes( link ), `Unused RichText link: ${link}` );
         } );
       }
       if ( this._nodes ) {
         Object.keys( this._nodes ).forEach( node => {
-          assert && allowLinksProperty.value && isNotDynamicStringTest && assert( usedNodes.includes( node ), `Unused RichText node: ${node}` );
+          assert && allowLinksProperty.value && !isStringTest && assert( usedNodes.includes( node ), `Unused RichText node: ${node}` );
         } );
       }
     }